09:20:19 <vincentfrancoise> #startmeeting watcher
09:20:19 <openstack> Meeting started Wed Dec 21 09:20:19 2016 UTC and is due to finish in 60 minutes.  The chair is vincentfrancoise. Information about MeetBot at http://wiki.debian.org/MeetBot.
09:20:20 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
09:20:22 <openstack> The meeting name has been set to 'watcher'
09:20:24 <vincentfrancoise> hi
09:20:38 <hidekazu> hello
09:20:50 <vincentfrancoise> Today's meeting will be driven by me
09:20:58 <tmetsch> hi
09:21:10 <vincentfrancoise> the agenda was made in 10mn so please tell me if you want to discuss any extra point
09:21:36 <vincentfrancoise> agenda for today #link https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#12.2F21.2F2016
09:21:52 <vincentfrancoise> #topic Announcements
09:22:04 <vincentfrancoise> Any annoucement?
09:22:32 <vincentfrancoise> ok so let's move on
09:22:33 <vincentfrancoise> #topic Review Action Items
09:23:26 <Yumeng> hi
09:24:11 <vincentfrancoise> #item https://review.openstack.org/#/c/377100/ Define grammar for workload characterization needs a new PS an answering previous comments
09:24:52 <vincentfrancoise> #info https://review.openstack.org/#/c/377100/ Define grammar for workload characterization needs a new PS an answering previous comments
09:25:36 <vincentfrancoise> is hvprash_ here?
09:26:25 <vincentfrancoise> #info https://review.openstack.org/#/c/401111/ needs a new PS
09:27:03 <vincentfrancoise> hanrong: I made a comment following what dtardivel said in a previous PS
09:27:11 <vincentfrancoise> did you see my comments?
09:27:20 <hanrong> Yea, I see
09:27:44 <hanrong> I am trying to using notification
09:28:13 <hanrong> I will update it later
09:28:51 <vincentfrancoise> hanrong: if you go with the notification option, you first need to implement the action notifications and that would mean you need https://blueprints.launchpad.net/watcher/+spec/action-versioned-notifications-api
09:29:29 <hanrong> ok , I will study it, thank you.
09:29:34 <vincentfrancoise> in order to publish the list of all actions along with their description
09:29:39 <vincentfrancoise> hanrong: ok great :)
09:30:02 <vincentfrancoise> #info https://review.openstack.org/#/c/398162/ needs reviews
09:30:49 <vincentfrancoise> who wants to review this spec?
09:31:04 <Yumeng__> +1
09:31:30 <vincentfrancoise> #action Yumeng_ review  https://review.openstack.org/#/c/398162/
09:32:26 <vincentfrancoise> #info https://review.openstack.org/#/c/391374/ There are many +1s but TODOs are still left in spec
09:33:10 <Yumeng__> Ok
09:33:22 <vincentfrancoise> see https://review.openstack.org/#/c/391374/5/specs/ocata/approved/audit-tag-vm-metadata.rst@264
09:34:04 <vincentfrancoise> I wasn't there last week
09:34:24 <vincentfrancoise> does anyone know if there was anything important said about this one last week>
09:34:26 <vincentfrancoise> ?
09:35:41 <vincentfrancoise> #info https://review.openstack.org/#/c/385871/ needs a new PS
09:35:57 <tmetsch> from last meeting: needs review
09:36:12 <vincentfrancoise> tmetsch: ok thanks
09:37:07 <vincentfrancoise> so can someone that already reviewed https://review.openstack.org/#/c/391374/ give their grain of salt on the todo's any the open questions left in the spec ?
09:38:04 <vincentfrancoise> ok so let's switch to Watcher reviews
09:38:18 <tmetsch> all US folks who volunteered to review afaik - so probably nobody here :-)
09:39:27 <vincentfrancoise> tmetsch: yeah possible but I didn't pay attention about it when quickly making the agenda :)
09:40:01 <vincentfrancoise> #info https://blueprints.launchpad.net/watcher/+spec/centralise-config-opts needs a new PS to remove _opts.py
09:40:50 <vincentfrancoise> pshedimb: are you there?
09:42:19 <jed56> vincentfrançoise will come back soon his client has crashed
09:42:49 <vincentfrancoise> I'm back
09:43:09 <Yumeng> :)
09:43:15 <vincentfrancoise> #action pshedimb add a new PS to remove _opts.py for https://blueprints.launchpad.net/watcher/+spec/centralise-config-opts
09:43:58 <vincentfrancoise> #info https://review.openstack.org/#/c/403718/ needs final reviews & merge
09:44:47 <vincentfrancoise> alexchadin: did you fix the issue raised by dtardivel on this one?
09:45:09 <alexchadin> vincentfrancoise: here is fix: https://review.openstack.org/#/c/403718/7..8/watcher/decision_engine/audit/base.py
09:46:04 <vincentfrancoise> #action vincentfrancoise review https://review.openstack.org/#/c/403718/
09:46:12 <vincentfrancoise> anyone else to review https://review.openstack.org/#/c/403718/ ?
09:46:48 <hanrong> +1
09:46:48 <Yumeng> +1
09:47:11 <vincentfrancoise> #action hanrong Yumeng review https://review.openstack.org/#/c/403718/
09:47:37 <vincentfrancoise> #info https://review.openstack.org/#/c/362730/ needs reviews
09:48:00 <vincentfrancoise> this one is around for quite a while now
09:49:23 <vincentfrancoise> I remember that I wanted to replace our current compute model with its graph based counterpart instead of having to maintain both since they are storing the same information
09:49:59 <tmetsch> that would be cool - but maybe that is sth for Pike?
09:50:09 <vincentfrancoise> but never actually took down to explain how/why
09:50:15 <vincentfrancoise> time*
09:50:46 <vincentfrancoise> tmetsch: sure this can be done in 2 stages
09:51:44 <tmetsch> I'd support that - for upcoming releases. For now a review of PS 15 would be great :-)
09:51:45 <vincentfrancoise> # action vincentfrancoise acabot create a BP for Pike to replace the current compute model with the graph based one
09:51:57 <vincentfrancoise> #action vincentfrancoise acabot create a BP for Pike to replace the current compute model with the graph based one
09:52:16 <vincentfrancoise> in the meantime, who wants to review this?
09:52:53 <vincentfrancoise> #action vincentfrancoise review https://review.openstack.org/#/c/362730/
09:53:06 <vincentfrancoise> #info https://review.openstack.org/#/c/406991/ needs a new PS
09:53:31 <vincentfrancoise> #info https://review.openstack.org/#/c/391383/ also needs a new PS
09:53:44 <vincentfrancoise> now for Watcher CLI
09:54:35 <vincentfrancoise> #info https://review.openstack.org/#/c/403716/ needs final reviews but will be merged after its dependency
09:55:04 <vincentfrancoise> #topic Blueprint/Bug
09:55:29 <vincentfrancoise> #info https://launchpad.net/watcher/+milestone/ocata-3 Watcher ocata-3 has started and finishes on January 23rd with 12 BPs to finish
09:56:04 <vincentfrancoise> so please focus on the BPs and bugs listed there
09:56:57 <vincentfrancoise> #info https://review.openstack.org/#/c/413079/ has been pushed by alexchadin and needs reviews (I missed it from earlier)
09:57:38 <vincentfrancoise> I personally won't have time myself to review this one this week
09:57:44 <vincentfrancoise> but maybe next week
09:57:55 <vincentfrancoise> who's willing to review it?
09:58:16 <vincentfrancoise> #topic open discussion
09:58:45 <vincentfrancoise> #link https://blueprints.launchpad.net/watcher/+spec/cluster-energy-saving
09:59:05 <vincentfrancoise> Yumeng you wanted to discuss this BP?
09:59:13 <hanrong> yes
09:59:34 <vincentfrancoise> I saw it was marked as superseded
09:59:40 <vincentfrancoise> (1mn left)
09:59:40 <hanrong> This bp need to discuss again
09:59:55 <hanrong> I add new description
10:00:03 <vincentfrancoise> ok
10:00:23 <vincentfrancoise> #action hanrong rewrite description for https://blueprints.launchpad.net/watcher/+spec/cluster-energy-saving  <https://blueprints.launchpad.net/watcher/+spec/cluster-energy-saving>
10:00:28 <vincentfrancoise> time's up
10:00:39 <vincentfrancoise> #endmeeting