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