14:00:19 #startmeeting watcher 14:00:19 Meeting started Wed Nov 15 14:00:19 2017 UTC and is due to finish in 60 minutes. The chair is alexchadin. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:00:20 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:22 The meeting name has been set to 'watcher' 14:00:40 Greetings! 14:00:44 Hi 14:00:47 started? 14:00:57 hi 14:01:02 hidekazu: just now 14:01:09 hi 14:01:38 Today we have the following agenda: https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#11.2F15.2F2017 14:01:49 #topic Announcements 14:02:03 Yumeng Bao has presented Project Update of Watcher #link https://www.youtube.com/watch?v=1FGb-uocfTw 14:02:25 Yumeng__: thanks a lot for your presentation, you've done it great! 14:03:08 Yumeng__: thanks for moderating forum physically. 14:03:47 :) I’m very happy to do that 14:04:39 Moreover, thanks for your feedback about using of Watcher #link https://etherpad.openstack.org/p/watcher-sydney-forum 14:05:57 suzhengwei: you've listed interesting things to discuss 14:06:24 I will talk with you about them tomorrow, I think 14:06:33 ok 14:06:46 suzhengwei: if you are able to, of course 14:07:07 i will make time tomorrow 14:07:50 any other announcements? 14:07:57 alexchadin: thanks for adding answers. i listed questions. 14:08:53 Yumeng__: what was the reaction of people on your presentation? 14:09:33 hidekazu: yeap, it would be useful to plan our roadmap for future releases 14:10:36 alexchadin: some new people interested in our projects 14:10:49 Not bad :) 14:11:12 Yumeng__: have you told them about our contacts or weekly meetings? 14:11:24 Several people come to ask questions and considering join us 14:11:59 Yes, I showed our IRC channel and meeting time 14:12:15 on prensentation 14:12:55 Yumeng__: oh, I haven't seen last slides 14:13:20 well, let's move on 14:13:31 #topic Review Action Items 14:13:34 Watcher specs 14:13:47 exclude locked instances needs to be updated #link https://review.openstack.org/#/c/506078/ 14:14:25 suzhengwei: we have discussed with you about locked property of VM that looks interesting 14:14:41 suzhengwei: please, update your spec with more details 14:14:59 sorry, I forgot to update the spec this. I will tomorrow 14:15:16 suzhengwei: thanks 14:15:27 #action suzhengwei update https://review.openstack.org/#/c/506078/ 14:15:40 Add Datasource Abstraction and Multi Datasource needs final core review #link https://review.openstack.org/#/c/508384/ 14:16:01 hidekazu: it would be good if you review it 14:16:16 alexchadin: yes, i will/ 14:17:00 hidekazu: thanks 14:17:16 #action hidekazu give final core review https://review.openstack.org/#/c/508384/ 14:17:32 cluster-maintenance-strategy needs to be updated #link https://review.openstack.org/#/c/464602/ 14:18:06 ok 14:18:12 suzhengwei: please, update your spec with your answers 14:18:49 suzhengwei: the more details you provide, the faster it is merged 14:18:50 yes, i will 14:18:54 suzhengwei: thanks 14:19:06 #action suzhengwei update https://review.openstack.org/#/c/464602/ 14:19:19 Add storage capacity balance Strategy needs new PS (Zuul issue) #link https://review.openstack.org/#/c/501659/ 14:21:03 Yumeng__: have you seen Zuul error? 14:22:42 Add audit scoper for storage data model spec needs new PS 14:22:49 #link https://review.openstack.org/#/c/517169/ 14:23:19 alexchadin: adisky_ will update and implement this spec. we discussed today. 14:24:05 alexchadin: because i joined customers project for about 40% from this month. 14:24:17 hidekazu: ok, next patch set will be from her? 14:24:38 alexchadin: yes. 14:24:53 hidekazu: fine 14:25:00 alexchadin: we discussed face to face in japan today 14:25:11 #action adisky_ update https://review.openstack.org/#/c/517169/ 14:25:16 alexchadin: yep, I have seen zuul error. Will check tomorrow 14:25:27 Yumeng__: thanks 14:25:47 #action Yumeng__ update https://review.openstack.org/#/c/501659/ 14:27:01 listen to all necessary notifications needs new PS #link https://review.openstack.org/#/c/508105/ 14:27:21 It adds two event notifications, is there any event we miss? 14:29:21 suzhengwei: I haven't all notifications in nova, that ones you have provided make sense to me 14:29:32 haven't seen* 14:30:09 suzhengwei: I've left some comments, everything else looks good to me 14:31:50 ok. 14:33:15 Watcher 14:33:24 improve 'Restrict existing strategies to their default scope' needs reviews #link https://review.openstack.org/#/c/504857/ 14:34:11 suzhengwei: it is the second commit regarding extend-node-status ? 14:34:27 suzhengwei: https://review.openstack.org/#/c/464557/ is it still actual? 14:35:33 yes 14:36:25 who will review https://review.openstack.org/#/c/504857/ ? 14:37:11 do they have dependency? 14:38:06 host maintaining depend it. 14:38:07 suzhengwei: ^ 14:39:08 ok, they are independent. 14:40:46 hidekazu: would you review it? 14:41:00 suzhengwei: do we really need https://review.openstack.org/#/c/464557/ ? I mean, isn't disabled_reason implemented by https://review.openstack.org/#/c/504857/ ? 14:41:02 suzhengwei: ok. 14:41:04 +1 14:42:43 #action hidekazu Yumeng__ review https://review.openstack.org/#/c/504857/ 14:42:49 maybe commit message is wrong or something. i will review including comment from alexchadin :) 14:43:18 option to rollback migrations when action_plan fails needs new PS #link https://review.openstack.org/#/c/517179/ 14:43:54 It need more comment, althought I prefer licanwei's. 14:44:50 #action suzhengwei update https://review.openstack.org/#/c/517179/ 14:45:00 Notifications Changes Multiple Global Efficacy needs reviews #link https://review.openstack.org/#/c/519299/ 14:45:23 I will review it, who else? :) 14:46:04 I will 14:46:46 #action suzhengwei alexchadin review https://review.openstack.org/#/c/519299/ 14:46:58 Add and identify excluded instances in compute CDM needs reviews #link https://review.openstack.org/#/c/504303/ 14:47:52 suzhengwei: is it still actual https://review.openstack.org/#/c/508108/ ? 14:48:07 yes 14:49:27 I think, you need to add unit tests at least 14:50:48 The commit is not finished ,https://review.openstack.org/#/c/508108/ 14:51:12 suzhengwei: then mark it as [WiP], please 14:51:28 Add storage capacity balance Strategy needs reviews #link https://review.openstack.org/#/c/516500/ 14:52:52 enough for reviewing? he updated four times. 14:53:09 three times. 14:53:43 hidekazu: since there is no [WiP] tag and unit tests are presented, I suppose we can review it 14:53:56 alexchadin: ok, i try 14:54:22 code includes volume migration :) 14:54:39 #action hidekazu review https://review.openstack.org/#/c/516500/ 14:54:50 hidekazu: then it should be familiar to you :) 14:55:03 alexchadin: yeap 14:55:05 Can't cancell CONTINUOUS audit needs reviews #link https://review.openstack.org/#/c/514540/ 14:55:29 who would like to review it? 14:56:12 add me 14:56:55 #action suzhengwei review https://review.openstack.org/#/c/514540/ 14:57:09 Add baremetal data model needs reviews #link https://review.openstack.org/#/c/482093/ 14:57:44 hidekazu: what about it? 14:58:21 alexchadin: yeah, i have to.. i will in two weeks. 14:58:29 hidekazu: thanks 14:58:43 #action hidekazu review https://review.openstack.org/#/c/482093/ 14:58:55 unfortunately, we run out of time 14:59:05 thank you for coming! 14:59:13 thanks. 14:59:20 see you next week! 14:59:21 bye 14:59:25 bye 14:59:45 #endmeeting