13:00:23 <alexchadin> #startmeeting watcher
13:00:23 <openstack> Meeting started Wed Apr 12 13:00:23 2017 UTC and is due to finish in 60 minutes.  The chair is alexchadin. Information about MeetBot at http://wiki.debian.org/MeetBot.
13:00:24 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
13:00:27 <openstack> The meeting name has been set to 'watcher'
13:00:36 <alexchadin> hi
13:00:41 <hidekazu> hi
13:00:44 <vincentfrancoise> o/
13:01:01 <alexchadin> Our agenda: https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#04.2F12.2F2017
13:01:14 <vincentfrancoise> will be back in 5mn sorry
13:01:50 <alexchadin> is there someone else?
13:02:41 <alexchadin> #topic Announcements
13:02:48 <alexchadin> #info Watcher & Watcher Client & Watcher Dashboard will be released for Pike-1 milestone this Thursday.
13:03:55 <alexchadin> We have done Gnocchi support, stale/invalid action plan and suspend state for audit
13:04:30 <alexchadin> congratulations to sanfern, licanwei and hidekazu!
13:04:45 <hidekazu> thanks everyone!
13:05:01 <alexchadin> and to everyone who reviewed and helped with these BPs
13:05:01 <licanwei> :)
13:05:12 <alexchadin> #info 3 of 5 P-1 targeted blueprints have been done, the rest will be retargeted to P-2 cycle.
13:05:30 <alexchadin> I had conversation with sballe_ and ostroverkhov
13:06:01 <alexchadin> They have agreed that their BPs should be moved to Pike-2 milestone
13:07:03 <alexchadin> is there any other announcements?
13:07:40 <alexchadin> well, move on
13:07:54 <sanfern> hello
13:08:01 <alexchadin> #topic Review Action Items
13:08:03 <alexchadin> hi sanfern
13:08:07 <alexchadin> Watcher specs
13:08:18 <alexchadin> Audit tag in VM Metadata needs a little fix and final core reviews #link https://review.openstack.org/#/c/391374/
13:09:27 <alexchadin> #action hvprash fix https://review.openstack.org/#/c/391374/
13:09:42 <alexchadin> Add spec for services-versioned-notifications-api bp should be updated in accordance with discussed changes #link https://review.openstack.org/#/c/445817/
13:10:18 <alexchadin> Vladimir is not here so I will give him details later
13:10:40 <alexchadin> #action alexchadin have a discussion with ostroverkhov about https://review.openstack.org/#/c/445817/
13:10:53 <alexchadin> Spec for noisy-neighbor-dashboard needs new PS #link https://review.openstack.org/#/c/427308/
13:11:37 <pksingh> Hello
13:11:40 <alexchadin> hi pksingh
13:12:35 <alexchadin> #action chrisspencer update https://review.openstack.org/#/c/427308/
13:12:48 <alexchadin> Noisy Neighbor Strategy needs reviews #link https://review.openstack.org/#/c/427308/
13:13:05 <alexchadin> #link https://review.openstack.org/#/c/398162/
13:14:09 <alexchadin> well seems hvprash won't be there for now
13:14:19 <alexchadin> who would like to review it?
13:15:15 <licanwei> me:)
13:15:28 <alexchadin> licanwei: thanks :)
13:15:37 <alexchadin> #action licanwei review https://review.openstack.org/#/c/398162/
13:15:59 <alexchadin> Support visualizing workload fingerprints needs new PS #link https://review.openstack.org/#/c/448394/
13:17:34 <alexchadin> Add spec for Cinder model integration needs new PS #link https://review.openstack.org/#/c/450173/
13:17:52 <hidekazu> alexchadin: yes i will do.
13:18:05 <alexchadin> hidekazu: vincentfrancoise left some comments for you
13:18:21 <alexchadin> #action hidekazu update https://review.openstack.org/#/c/450173/
13:19:00 <alexchadin> Build baremetal data model in watcher. Yumeng provided new PS #link https://review.openstack.org/#/c/448016/
13:19:25 <alexchadin> dtardivel is already reviewing it. Who would like to take it too?
13:20:02 <licanwei> +1
13:20:05 <hidekazu> alexchadin: i will review.
13:20:34 <hidekazu> alexchadin: because data model, too :)
13:20:58 <alexchadin> #action licanwei hidekazu review https://review.openstack.org/#/c/448016/
13:21:08 <alexchadin> hidekazu licanwei thanks a lot!
13:21:17 <alexchadin> Audit tag in VM Metadata needs final core review #link https://review.openstack.org/#/c/391374/
13:21:40 <alexchadin> oups
13:22:04 <alexchadin> duplication :)
13:22:15 <alexchadin> Define grammar for workload characterization needs reviews #link https://review.openstack.org/#/c/377100/
13:22:57 <alexchadin> we need more reviews here
13:24:48 <alexchadin> #action alexchadin review https://review.openstack.org/#/c/377100/
13:24:57 <alexchadin> Add new actions "power on" and "power off" in Watcher needs reviews #link https://review.openstack.org/#/c/435436/
13:25:46 <alexchadin> hidekazu and licanwei: will you continue review this spec?
13:26:06 <licanwei> +1
13:26:13 <hidekazu> alexchadin: ok ,i will review
13:26:33 <alexchadin> #action licanwei hidekazu review https://review.openstack.org/#/c/435436/
13:26:40 <alexchadin> Watcher
13:27:02 <alexchadin> Reduced the code complexity needs code reviews #link https://review.openstack.org/#/c/441873/
13:29:04 <alexchadin> Noisy Neighbor Strategy needs new PS #link https://review.openstack.org/#/c/454359/
13:30:04 <alexchadin> licanwei hidekazu I will add you
13:30:28 <alexchadin> as well as watcher-core group
13:30:28 <licanwei> ok
13:31:06 <alexchadin> Adding tempest test that executes outlet_temp_control strategy. needs new PS #link https://review.openstack.org/#/c/439184/
13:32:20 <alexchadin> #action chrisspencer update https://review.openstack.org/#/c/439184/
13:32:36 <alexchadin> Added tempest test for workload_stabilization needs final core reviews #link https://review.openstack.org/#/c/393355/
13:33:38 <alexchadin> it would be good if someone would test it on Devstack
13:36:21 <alexchadin> Add action description notification needs reviews #link https://review.openstack.org/#/c/454638/
13:36:45 <alexchadin> seems there is merge conflict
13:36:58 <alexchadin> licanwei have you got notification?
13:37:16 <licanwei> yes
13:38:36 <alexchadin> #action licanwei fix merge conflict on https://review.openstack.org/#/c/454638/
13:38:45 <alexchadin> #action alexchadin review https://review.openstack.org/#/c/454638/
13:38:55 <hidekazu> licanwei: this patch is partial implementation?
13:38:55 <alexchadin> Add action description needs reviews #link https://review.openstack.org/#/c/450720/
13:39:15 <licanwei> no
13:39:27 <hidekazu> licanwei: ok,thanks.
13:40:11 <hidekazu> alexchadin: go ahead, please.
13:40:30 <alexchadin> Watcher CLI & Dashboard
13:40:41 <alexchadin> Add a button to create audit template in audit creat form needs core reviews #link https://review.openstack.org/#/c/455699/
13:41:08 <alexchadin> There are two fixes from Yumeng
13:41:48 <alexchadin> Who is familiar with Horizon?
13:43:27 <licanwei> I'll try :)
13:44:09 <alexchadin> #action licanwei review https://review.openstack.org/#/c/455699/ https://review.openstack.org/#/c/455679/
13:44:19 <alexchadin> #topic Blueprint/Bug Review and Discussion
13:44:46 <alexchadin> exception when running 'watcher actionplan start XXX' #link https://review.openstack.org/#/c/450555/
13:45:07 <licanwei> say alexchadin : this commit should be merged ASAP.
13:45:42 <licanwei> Now the actionplan start cli cmd run exception
13:46:15 <hidekazu> alexchadin: licanwei: i may hit this bug yesterday.
13:47:30 <alexchadin> licanwei: why have you add self.action on line 49?
13:47:39 * vincentfrancoise back...
13:47:53 <alexchadin> vincentfrancoise: it was long 5 minutes ;)
13:48:02 <vincentfrancoise> yeah, ask dtardivel about it :p
13:48:32 <licanwei> this can triger notification
13:48:59 <licanwei> if actionplan is null ,no notifican emit
13:49:10 <licanwei> notification
13:49:47 <alexchadin> vincentfrancoise: will you review it ASAP? https://review.openstack.org/#/c/450555/
13:49:52 <licanwei> this can hit the bug throug unit test
13:50:53 <vincentfrancoise> alexchadin: ok
13:51:07 <alexchadin> vincentfrancoise: thanks a lot!
13:51:19 <alexchadin> #action vincentfrancoise review https://review.openstack.org/#/c/450555/
13:51:33 <alexchadin> #topic Open discussions
13:51:34 <vincentfrancoise> alexchadin: done
13:51:57 <vincentfrancoise> alexchadin: short fix = quick review ;)
13:52:13 <alexchadin> vincentfrancoise: then I W+1 it
13:52:37 <vincentfrancoise> alexchadin: shouldn't this be another core's job?
13:53:07 <vincentfrancoise> alexchadin: oh sorry misread you
13:53:17 <vincentfrancoise> alexchadin: yeah please W+1 it ;)
13:53:23 <alexchadin> There is few people today
13:53:42 <alexchadin> sballe_ said that she will have some things to do
13:54:37 <alexchadin> BTW, I'm happy that we have achieved P-1 milestone with 3 of 5 BPs
13:55:04 <alexchadin> 2 of them should be merged soon, I hope next week
13:55:56 <vincentfrancoise> alexchadin: good :)
13:56:03 <alexchadin> I mean, the rest ones
13:56:45 <vincentfrancoise> alexchadin: I missed the review parts but if there's some critical specs to review, add them as actions to me so I can see them in the logs
13:57:29 <alexchadin> vincentfrancoise: no worries, I will add some of mentioned specs to you a little bit later
13:58:11 <alexchadin> so, if nobody have questions, I would suggest to end this meeting
13:58:49 <alexchadin> thank you for coming, bye
13:58:58 <hidekazu> alexchadin: thanks
13:59:14 <hidekazu> good night ;)
13:59:19 <vincentfrancoise> bye :)
13:59:26 <alexchadin> #endmeeting