14:00:03 <alexchadin> #startmeeting watcher 14:00:04 <openstack> Meeting started Wed Jul 12 14:00:03 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:05 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:08 <openstack> The meeting name has been set to 'watcher' 14:00:12 <alexchadin> hi 14:00:16 <Yumeng__> Hi 14:01:06 <alexchadin> when it starts at 14:00 utc, there are less people than at 13:00 utc 14:01:17 <alexchadin> hi hidekazu suzhengwei 14:01:21 <alexchadin> hi Yumeng__ 14:01:23 <suzhengwei> hi 14:01:55 <alexchadin> today we have the following agenda: https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#07.2F12.2F2017 14:02:12 <alexchadin> #topic Announcements 14:02:22 <alexchadin> watcher repo has prepared docs. watcher-dashboard and python-watcherclient are still on the way. 14:02:52 <alexchadin> we've made almost all required steps to reach new docs standards 14:03:15 <alexchadin> Yumeng__ thank you for your help with doc-migration 14:03:38 <lakerzhou> lakerzhou 14:03:39 <suzhengwei> hi, lakerzhou 14:03:44 <Yumeng__> My pleasure . I noticed that we have lots of bugs in dashboard 14:04:02 <lakerzhou> Hi, Suzhengwei 14:04:25 <Yumeng__> We should fix those bugs and then we can be able to merge doc-migration in watcher-dashboard 14:04:38 <alexchadin> Yumeng__: yeah, I'm working on it 14:05:28 <Yumeng__> alexchadin: great! 14:05:30 <alexchadin> Three presentations have been submitted #link https://etherpad.openstack.org/p/watcher-Sydney-meetings 14:05:48 <alexchadin> Don't hesitate to submit your own 14:06:23 <alexchadin> two of three are in the same category 14:06:29 <hidekazu_> alexchadin: sorry i missed write to etherpad. 14:06:36 <adisky__> Hi sorry for late 14:06:41 <alexchadin> hi adisky__ ! 14:06:55 <alexchadin> hidekazu_: no worries, just don't forget to do it ;) 14:07:11 <Yumeng__> And I have one idea to share with you guys! 14:07:35 <lakerzhou> Yumeng is on all three talks! 14:08:01 <hidekazu_> Yumeng__: three is max. 14:08:29 <alexchadin> I hope they won't be at the same time 14:08:36 <Yumeng__> yep 14:08:44 <alexchadin> as it was in Boston :) 14:09:06 <alexchadin> two watcher presentations at the same time 14:09:15 <alexchadin> some kind of magic 14:09:20 <Yumeng__> Sorry to disturb you. I noticed that recently we have quite a few patches waiting for review but only have limited developers to do the review. 14:10:02 <Yumeng__> So I would suggest that we give different priorities to the patches and review according to their priorities. 14:10:18 <Yumeng__> Here I proposed a example on etherpad, please take a look 14:10:29 <Yumeng__> https://etherpad.openstack.org/p/watcher-review-priority 14:10:31 <hidekazu_> Yumeng__: personally i am reviewing for pike-3 14:11:08 <Yumeng__> hidekazu_: Good! 14:11:23 <alexchadin> Yumeng__: I try to review as much as I can, but there are a lot of deals with Foundation's crew 14:11:27 <hidekazu_> Yumeng__: now only milestone is important. 14:11:33 <alexchadin> hidekazu_: +1 14:12:09 <hidekazu_> https://launchpad.net/watcher/+milestone/pike-3 14:12:12 <alexchadin> if all milestone patches needs new PSs, we may review other ones 14:12:15 <Yumeng__> hidekazu_ :+1. It's just a personal idea! Leave any comment on etherpad. 14:13:18 <Yumeng__> alexchadin: +1. I remember Vincent suggested that we give him a priority about reviewing. 14:14:06 <alexchadin> so 14:14:13 <alexchadin> any other announcements? 14:14:52 <suzhengwei> everybody, https://review.openstack.org/#/c/468261/ this spec is important to watcher. It need your review. 14:16:34 <alexchadin> suzhengwei: IMHO, you should attract an attention of nova's folks to be sure that it's not -2 commit 14:16:44 <hidekazu_> suzhenwei: this is nova spec, you need nova help. 14:17:09 <hidekazu_> s/suzhenwei/suzhengwei/ 14:17:23 <alexchadin> suzhengwei: we will review it, but I personally will do it only when nova team are agreed with it 14:18:23 <alexchadin> suzhengwei: cause there may be 1000 reasons why it's not correlate with nova policies ;) 14:18:47 <suzhengwei> yes, but it infect watcher. It need promote. 14:19:24 <alexchadin> suzhengwei: I recommend you to contact with ed leafe 14:19:37 <suzhengwei> ok 14:20:02 <alexchadin> suzhengwei: he is our liaison with nova team 14:20:15 <sballe_> hi 14:20:19 <alexchadin> hi sballe_ 14:20:44 <alexchadin> well, let's move on 14:21:05 <alexchadin> #topic Review Action Items 14:21:07 <alexchadin> watcher 14:21:10 <alexchadin> specs 14:21:19 <alexchadin> Add notfications for actionplan cancel needs reviews #link https://review.openstack.org/#/c/482902/ 14:22:06 <alexchadin> adisky__: is it ready to be reviewed? 14:22:36 <adisky__> alexchadin: yes 14:23:25 <alexchadin> adisky__: please take a look at it: https://review.openstack.org/#/c/445817/7/specs/pike/approved/services-versioned-notifications-api.rst 14:23:59 <alexchadin> adisky__: you should show notification structures at least 14:24:12 <adisky__> alexchadin: ok, I will add 14:24:19 <alexchadin> adisky__: thank you 14:24:29 <alexchadin> adisky__: and tag me as reviewer 14:25:00 <adisky__> alexchadin: I thought, it is same as previous notifications , so no need to add structure 14:25:12 <alexchadin> Add Volume migrate action spec needs final core review #link https://review.openstack.org/#/c/468817/ 14:26:14 <alexchadin> adisky__: anyway, it is standalone spec and should be prepared accordingly 14:26:44 <alexchadin> hidekazu_: diagram is great! 14:26:47 <adisky__> alexchadin: I will add 14:27:07 <alexchadin> #action adisky__ update https://review.openstack.org/#/c/482902/ 14:27:48 <hidekazu_> Add Volume migrate action spec needs core reviewer other than me. 14:28:27 <alexchadin> hidekazu_: I will review it 14:28:41 <hidekazu_> alexchadin: thanks! 14:28:58 <alexchadin> hidekazu_: I'll do it tomorrow, ping me to remind please 14:29:19 <hidekazu_> alexchadin: ok. 14:29:26 <alexchadin> #action alexchadin review https://review.openstack.org/#/c/468817/ 14:29:38 <alexchadin> Specs for Saving Energy Strategy needs reviews #link https://review.openstack.org/#/c/476416 14:30:12 <alexchadin> how would like to review it? 14:30:18 <alexchadin> who* 14:30:32 <hidekazu_> alexchadin: i first reviewed code, if new ps of code proposed, i will review spec. 14:31:21 <hidekazu_> alexchadin: code is readable than spec about saving energy now. 14:32:26 <suzhengwei> agree 14:32:52 <alexchadin> suzhengwei, hidekazu_ I will tag you as reviewers, ok? 14:33:06 <hidekazu_> alexchadin: ok. 14:33:10 <suzhengwei> ok, i will give new review. 14:33:48 <alexchadin> #action suzhengwei hidekazu_ review https://review.openstack.org/#/c/476416 14:33:59 <alexchadin> update action description needs to be updated #link https://review.openstack.org/#/c/459529/ 14:33:59 <vincentfrancoise> hi everyone 14:34:03 <alexchadin> hi vincentfrancoise 14:34:24 <alexchadin> unfortunately, licanwei isn't here today 14:34:46 <alexchadin> [WIP] Spec for multi-datasource support] needs reviews #link https://review.openstack.org/#/c/467102 14:34:47 <hidekazu_> vincentfrancoise: hi 14:35:16 <adisky__> Will review 14:35:17 <alexchadin> vincentfrancoise: seems it's your commit to review ;) 14:35:40 <alexchadin> adisky__: nice! 14:35:49 <vincentfrancoise> didn't have time to do any meaningful review since it's a short week in France (14th is a bank holiday) 14:35:59 <alexchadin> #action adisky__ review https://review.openstack.org/#/c/467102 14:36:56 <alexchadin> vincentfrancoise: I'll tag you as reviewer, ok? 14:37:49 <hidekazu_> French also has bank holiday.. 14:37:50 <vincentfrancoise> alexchadin: yup 14:37:54 <hidekazu_> sorry. 14:38:15 <suzhengwei> I will review Spec for multi-datasource support,tag me too. 14:38:22 <alexchadin> suzhengwei: thanks! 14:38:29 <lakerzhou> thanks suzhengwei 14:38:36 <alexchadin> #action suzhengwei vincentfrancoise review https://review.openstack.org/#/c/467102 14:38:37 <suzhengwei> welcome 14:38:48 <alexchadin> add name for audit needs new PS #link https://review.openstack.org/#/c/474865/ 14:40:08 <alexchadin> suzhengwei: contact to licanwei when he is available 14:40:18 <suzhengwei> ok 14:41:00 <alexchadin> Support visualizing workload fingerprints needs reviews #link https://review.openstack.org/#/c/448394/ 14:42:20 <alexchadin> suzhengwei: I will investigate your commit 14:42:35 <alexchadin> extend-node-status needs reviews #link https://review.openstack.org/#/c/464549/ 14:43:02 <alexchadin> adisky__: could you please review it again? 14:43:52 <Yumeng__> alexchadin: add me please. Sorry for late review. 14:44:11 <alexchadin> #action Yumeng__ review https://review.openstack.org/#/c/464549/ 14:44:20 <alexchadin> watcher 14:44:25 <alexchadin> New cron type for audit interval will be updated soon 14:44:36 <alexchadin> Cinder model integration needs reviews #link https://review.openstack.org/#/c/450642/ 14:44:43 <alexchadin> it's hard to review 14:45:23 <adisky__> alexchadin: will review extend node status 14:45:38 <hidekazu_> alexchadin: merge and fix is one way ;) 14:45:51 <alexchadin> #action adisky__ review https://review.openstack.org/#/c/464549/ 14:47:09 <vincentfrancoise> for the Cinder model IMHO we should get it merged as-is 14:47:18 <hidekazu_> alexchadin: no harm because of not part of core framework. 14:47:43 <alexchadin> I agree 14:47:59 <vincentfrancoise> I give it a +2 then 14:48:58 <alexchadin> +1 W 14:49:08 <alexchadin> Enable migration to rely on nova-scheduler needs final core review #link https://review.openstack.org/#/c/476369/ 14:49:39 <alexchadin> Yumeng__: will you take it? 14:50:10 <Yumeng__> alexchadin: yes no pb 14:50:28 <alexchadin> #action Yumeng__ review https://review.openstack.org/#/c/476369/ 14:50:37 <alexchadin> Get load of every available host by gathering instances load needs reviews #link https://review.openstack.org/#/c/482780/ 14:50:52 <alexchadin> suzhengwei: have you seen my comment? 14:51:06 <suzhengwei> yes. 14:51:49 <alexchadin> suzhengwei: we have discussed it when this strategy was in development 14:52:19 <alexchadin> Add action description notification needs new PS #link https://review.openstack.org/#/c/454638/ 14:52:34 <suzhengwei> I have noticed the discusion 14:52:49 <alexchadin> sorry, licanwei hasn't updated spec 14:53:10 <alexchadin> Abort operation for live migration needs final core review #link https://review.openstack.org/#/c/474568/ 14:53:58 <Yumeng__> alexchadin: +1 14:54:14 <alexchadin> #action Yumeng__ review https://review.openstack.org/#/c/474568/ 14:55:07 <alexchadin> extend-node-status needs reviews #link https://review.openstack.org/#/c/464557/ 14:55:18 <alexchadin> I propose to review implementation after spec is merged 14:55:32 <alexchadin> Watcher CLI & Dashboard 14:55:40 <alexchadin> Move existing content into the new standard structure needs reviews #link https://review.openstack.org/#/c/478342/ 14:56:18 <alexchadin> please, don't forget to review python-watcherclient and watcher-dashboard commits 14:56:29 <hidekazu_> +1 14:56:53 <suzhengwei> <alexchadin>https://review.openstack.org/#/c/479257/ this need more review. 14:57:39 <hidekazu_> alexchadin: W+1 14:57:41 <alexchadin> suzhengwei: yeap 14:58:01 <alexchadin> who would like to review https://review.openstack.org/#/c/479257/? 14:58:55 <alexchadin> hidekazu_: please take it 14:59:07 <hidekazu_> ok 14:59:08 <adisky__> I will also review 14:59:14 <suzhengwei> thx 14:59:20 <alexchadin> #action adisky__ hidekazu_ review https://review.openstack.org/#/c/479257/ 14:59:22 <alexchadin> well 14:59:28 <alexchadin> thank you for coming! 14:59:34 <alexchadin> our time is running out 14:59:42 <alexchadin> we've made great job! 14:59:54 <alexchadin> hope to see you next time ;) 15:00:03 <suzhengwei> see you 15:00:07 <Yumeng__> See you 15:00:10 <alexchadin> bye 15:00:16 <adisky__> bye 15:00:16 <hidekazu_> bye 15:00:17 <alexchadin> #endmeeting