08:01:37 #startmeeting vitrage 08:01:38 Meeting started Wed May 24 08:01:37 2017 UTC and is due to finish in 60 minutes. The chair is ifat_afek. Information about MeetBot at http://wiki.debian.org/MeetBot. 08:01:39 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 08:01:41 The meeting name has been set to 'vitrage' 08:01:45 Hi :-) 08:01:52 \o/ 08:03:06 Hi :) 08:03:11 hi 08:03:44 Hi 08:03:46 hi 08:04:14 Today’s agenda: 08:04:19 • Status and Updates 08:04:26 • Open Discussion 08:04:32 #topic Status and Updates 08:04:42 I found and fixed a bug in mark-down 08:04:52 Apparently there was no test for that… the latest changes in Vitrage-ID broke the mark-down action, and we didn’t noticed. I fixed the problem and added tests. 08:05:37 While working on that, idan_hefetz and I realized that the use of IS_REAL_VITRAGE_ID flag is not clear. doffek said he would change it (not sure when, it’s a matter of prioritis) 08:06:46 I also started thinking about the integration with Mistral. I’m not sure is what the best way to do it yet. Will update when I have a more clear idea 08:06:55 That’s it for me 08:07:25 I understand mistral is a workflow engine. How are vitrage going to use it? 08:07:45 A new action type ? Or something else? 08:07:56 That’s one of the questions 08:08:05 I was thinking of adding something like: 08:08:11 action: external 08:08:18 type: mistral 08:08:28 parameters: …. dictionary .... 08:08:57 But - I’m not sure if I should use the existing mechanism of notifiers for that. Or just call Mistral directly from the action 08:09:21 Using the notifiers seems like an abuse. Becuase only the Mistral notifier should care about the Mistral actions 08:09:45 And BTW - I want to make it generic, so adding a Congress action later on would be straight forward 08:10:18 Another option is that Mistral notifier will receive a notification from Vitrage graph, and decide what to do. This would require a new configuration file for Mistral notifier 08:10:44 Or - just pass the notification to Mistral via the message bus. And let Mistral workflow register on the wanted notification 08:11:03 These are my initial thoughts… and in any case, the implementation should be quite simple 08:11:29 Got it 08:11:38 (since I already managed to execute a Mistral workflow from Vitrage as part of the POC that was presented in Boston) 08:12:05 If you have any thoughts or ideas, I’ll be happy to hear them. Now, later, or in the mailing list :-) 08:13:24 Who wants to update next? 08:14:03 May I? 08:14:13 Sure 08:15:06 Entity equivalence is under review, thanks for all the comments 08:15:30 It is currently a combination of refactoring and new features 08:16:07 If someone wants a separate view of the refactoring, please check https://review.openstack.org/#/c/462532/ 08:16:39 You sent the abandoned issue 08:16:53 This patch does not include anything of equivalence, just an optimization of scenario repository 08:17:01 OK 08:17:06 yes 08:17:10 Because I thought Alexey has finished reviewing 08:17:25 And he is confused with the two patches, so I combined the two 08:17:29 And abandon one 08:17:37 I think it’s a good idea to combine them 08:17:54 I hope we’ll manage to finish the review process soon 08:18:00 Thanks. Another thing is about osprofiler 08:18:02 I think it’s a great improvement 08:18:21 I am looking into it for doctor project and it looks also valuable for vitrage 08:18:22 Which is MitmStack? something else? I don’t remember… 08:18:35 osprofiler 08:18:37 #link https://docs.openstack.org/developer/osprofiler/index.html 08:18:50 #link https://www.youtube.com/watch?v=57fTdE1hjeo&t=1223s 08:19:15 Right, now I remember that I’ve heard about it 08:19:23 Sounds like a good idea to use it in Vitrage 08:19:50 So did you start working on that? 08:20:17 I'm working on using it in doctor project. I will consider it for vitrage after OPNFV summit 08:20:30 Sounds good 08:20:42 Some other colleague in ZTE is also interested in adding trace point for vitrage for performance evaluation 08:21:06 I think this is the right tool rather than inventing another wheel 08:21:18 This could definitely help us 08:21:20 Currently, nova, glance, cinder and many openstack services has already integrated it 08:21:32 Hello Streptacocus :) 08:21:43 Hi Alexey! 08:21:47 I think it won't be too difficult. That's all 08:22:06 yujunz: Thanks 08:22:24 alexey_weyl: do you have any updates? ;-) 08:23:26 Wow, good question, let me see. 08:25:21 I haven't had too much time to work due to me being sick, but I have started to talk with people about some code refactoring in our constants so we will have a unified names 08:26:04 Good idea 08:27:14 Speaking of constant name, we need also think about some property name in template. 08:27:24 Like what? 08:27:37 Maybe a unified namespace `vitrage_` would be good for things like `is_deleted` 08:27:59 I think that’s alexey_weyl’s proposal, more or less 08:28:27 To add a vitrage_ prefix to all of the Vitrage properties, to avoid confusion 08:28:57 Yujun we all think a like.. 08:29:29 this is a refactoring that I have already strated to make and hopefully it will be done will the end of week.. 08:29:46 If you feel good, that is ;-) 08:29:57 hhhhhhhh, yeah :) 08:30:46 Any other update? alexey_weyl or someone else? 08:32:32 #topic Open Discussion 08:33:05 We need to decide whether we want to join the PTG in Denver in September. If enough people are interested, I will ask for a room for the design sessions 08:33:26 I sent an email to the OpenStack mailing list. Waiting for responses. 08:33:50 Any other issue you would like to discuss?\ 08:34:05 Nope 08:35:08 So let’s end the meeting 08:35:12 Bye :-) 08:35:16 bye 08:35:22 bye 08:35:29 bye 08:35:47 #endmeeting