09:00:38 <acabot> #startmeeting watcher
09:00:39 <openstack> Meeting started Wed Sep 28 09:00:38 2016 UTC and is due to finish in 60 minutes.  The chair is acabot. Information about MeetBot at http://wiki.debian.org/MeetBot.
09:00:40 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
09:00:42 <openstack> The meeting name has been set to 'watcher'
09:00:44 <acabot> o/
09:00:45 <vincentfrancoise> o/
09:01:08 <dtardivel> hi
09:01:34 <licanwei> hi
09:01:37 <tmetsch> hi
09:02:12 <acabot> agenda for today https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#09.2F28.2F2016
09:02:33 <acabot> #topic Announcements
09:02:52 <acabot> #info please complete topics list in barcelona summit etherpad
09:02:59 <acabot> #link https://etherpad.openstack.org/p/watcher-ocata-design-session
09:03:15 <acabot> tmetsch : do you plan to join us in Barcelona ?
09:03:31 <tmetsch> acabot: not 100% sure yet.
09:03:38 <acabot> ok
09:04:16 <acabot> licanwei : please complete the topics list in the etherpad
09:04:44 <acabot> #info Watcher Newton final release will be submitted today
09:05:10 <acabot> we will have a final release for watcher-pythonclient and watcher-dashboard as well
09:05:36 <acabot> please test it asap, it still time to submit bugs
09:05:51 <acabot> any other announcement ?
09:06:14 <acabot> #topic Review Action Items
09:06:22 <acabot> Watcher specs
09:06:30 <acabot> Resize instance base on workload needs reviews from core
09:06:39 <acabot> #link https://review.openstack.org/#/c/359690/
09:06:51 <acabot> licanwei : is this ready to be reviewed ?
09:07:15 <diga_> o/
09:07:18 <diga_> got late
09:07:23 <alexchadin> o/
09:07:26 <licanwei> acabot:ok
09:07:26 <acabot> diga_ : no pb
09:07:48 <acabot> #action sballe jwcroppe hvprash acabot review https://review.openstack.org/#/c/359690/
09:07:57 <acabot> Limit concurrent actions invoked by Watcher needs a new PS
09:08:04 <acabot> #link https://review.openstack.org/#/c/348049/
09:08:13 <acabot> #action jwcroppe add a new PS for https://review.openstack.org/#/c/348049/
09:08:24 <acabot> automatic triggering of action plans for AUDIT needs a new PS
09:08:31 <acabot> #link https://review.openstack.org/#/c/342473/
09:08:32 <diga_> acabot: yes
09:08:48 <acabot> diga_ : I'm concerned about the speed of this spec...
09:09:10 <diga_> acabot: I will make sure, it will get merged till next meeting
09:09:20 <acabot> diga_ : do you want me to take it over and then you will be able to start implementing ?
09:09:54 <acabot> diga_ : I dont think its ready for core reviews
09:10:05 <diga_> acabot: yes, I need to update it
09:10:17 <acabot> diga_ : could you please update it before EOW ?
09:10:22 <diga_> acabot: okay
09:10:40 <acabot> #action diga_ add a new PS for https://review.openstack.org/#/c/342473/
09:10:51 <acabot> Add team priorities for Ocata needs a new PS with BP split
09:10:59 <acabot> #link https://review.openstack.org/#/c/372528/
09:11:12 <acabot> #action acabot add a new PS for https://review.openstack.org/#/c/372528/
09:11:19 <acabot> Watcher
09:11:26 <acabot> Add Audit Scope Handler needs reviews
09:11:32 <acabot> #link https://review.openstack.org/#/q/status:open++topic:bp/define-the-audit-scope
09:11:52 <acabot> dtardivel is already working on it
09:12:09 <acabot> licanwei : do you want to review it ?
09:12:24 <licanwei> ok
09:13:15 <acabot> #action licanwei review https://review.openstack.org/#/q/status:open++topic:bp/define-the-audit-scope
09:13:38 <acabot> Graph cluster model implementation needs reviews
09:13:45 <acabot> #link https://review.openstack.org/#/c/362730/
09:14:01 <dtardivel> acabot: I'm revewing
09:14:07 <acabot> tmetsch : do you follow Kevin work on this ?
09:14:13 <tmetsch> yes
09:14:24 <acabot> #action dtardivel review https://review.openstack.org/#/q/status:open++topic:bp/define-the-audit-scope
09:14:42 <acabot> tmetsch : vincentfrancoise started reviewing it
09:15:11 <tmetsch> thanks for that - I'll work with Kevin and address the issue.
09:15:41 <vincentfrancoise> tmetsch: yeah I started reviewing it yesterday and there were a few points that were bugging me
09:16:21 <vincentfrancoise> I will address them in the review and I will post probably post them by EOD
09:16:43 <acabot> #action vincentfrancoise add questions on https://review.openstack.org/#/c/362730/
09:16:47 <tmetsch> vincentfrancoise: cool thanks!
09:17:06 <acabot> #action tmetsch add a new PS for https://review.openstack.org/#/c/362730/ answering comments and questions
09:17:16 <acabot> Add service supervisor needs reviews
09:17:24 <acabot> #link https://review.openstack.org/#/q/status:open+project:openstack/watcher+branch:master+topic:bp/watcher-service-list
09:17:38 <acabot> licanwei : do you want to review this as well ?
09:17:47 <licanwei> acabot: np
09:17:52 <dtardivel> I can review it as well
09:18:11 <acabot> #action licanwei dtardivel review https://review.openstack.org/#/q/status:open+project:openstack/watcher+branch:master+topic:bp/watcher-service-list
09:18:23 <acabot> Added Tempest API tests for /scoring_engines needs reviews
09:18:29 <acabot> #link https://review.openstack.org/#/c/371558/
09:19:31 <acabot> anyone likes to review tempest tests ?
09:19:51 <acabot> its a small PS ;-)
09:20:15 <licanwei> me
09:20:22 <acabot> thx
09:20:32 <acabot> #action licanwei review https://review.openstack.org/#/c/371558/
09:20:48 <acabot> lets move to Watcher CLI
09:21:00 <acabot> Added support for Client creation from KS session needs reviews
09:21:06 <acabot> #link https://review.openstack.org/#/c/374286/
09:21:42 <acabot> it would be great to have this merge for Newton
09:21:49 <acabot> but its a large PS...
09:22:41 <acabot> anyone familiar with keystone here ?
09:22:44 <dtardivel> +1
09:23:05 <acabot> #action dtardivel review https://review.openstack.org/#/c/374286/
09:23:06 <acabot> thx
09:23:12 <acabot> Watcher dashboard
09:23:19 <acabot> Added action details needs reviews
09:23:27 <acabot> #link https://review.openstack.org/#/c/377699/
09:23:27 <vincentfrancoise> this is essentially a big refactoring that allows us to simplify the use of python-watcherclient as an SDK instead of a shell command
09:23:59 <acabot> we really need to have the action details merged before the final release
09:24:05 <acabot> who can review it today ?
09:24:24 <dtardivel> +1
09:24:38 <vincentfrancoise> The idea is to make it behave like other projects to reduce the difficulty for people already used to other OS clients
09:24:43 <acabot> #action dtardivel review https://review.openstack.org/#/c/377699/
09:25:02 <acabot> thanks vincentfrancoise for the explanations
09:25:15 <acabot> #topic Blueprint/Bug Review and Discussion
09:25:30 <acabot> #info Improve the storage of the ActionPlan has many dependencies and should be started asap
09:25:36 <acabot> #link https://blueprints.launchpad.net/watcher/+spec/planner-storage-action-plan
09:25:52 <acabot> licanwei : this BP has been taken by jinquan
09:26:02 <acabot> licanwei : I'd like to start working on spec now
09:26:31 <licanwei> acabot: ok
09:26:40 <acabot> licanwei : do you know if he can leave the BP ?
09:26:57 <acabot> licanwei : or if he already started working a on spec ?
09:27:04 <licanwei> I don't know
09:27:24 <acabot> ok I will ask him on launchpad
09:27:43 <acabot> #action acabot ask jinquan to leave the BP https://blueprints.launchpad.net/watcher/+spec/planner-storage-action-plan
09:28:04 <licanwei> I can ask him
09:28:19 <acabot> ok
09:28:27 <acabot> #link https://launchpad.net/watcher/+milestone/newton-rc-final
09:28:40 <acabot> we still have 1 bug open
09:28:47 <acabot> #link https://bugs.launchpad.net/bugs/1620225
09:28:48 <openstack> Launchpad bug 1620225 in watcher "missing doc section to add notifications when building a new cluster data model collector" [Medium,Triaged]
09:29:15 <acabot> vincentfrancoise dtardivel : can we fix it before the release ?
09:29:54 <dtardivel> acabot: We already added a section in the configuration section ...;
09:30:22 <vincentfrancoise> acabot: doesn't it correspond to http://docs.openstack.org/developer/watcher/dev/plugin/cdmc-plugin.html#add-new-notification-endpoints ?
09:30:45 <vincentfrancoise> acabot: in which case we can close this bug straight away :)
09:31:07 <acabot> vincentfrancoise : :-) I think you can
09:31:29 <acabot> #topic Open discussions
09:31:56 <acabot> I forgot to mention that slides for Watcher talk in Barcelona are open to reviews
09:32:05 <acabot> #link https://drive.google.com/file/d/0B_YlsOGofvCCZFc3UnZ1MkxBNG8/view?ts=57eaab73
09:32:34 <acabot> #undo
09:32:35 <openstack> Removing item from minutes: <ircmeeting.items.Link object at 0x7f66f52ef750>
09:32:38 <acabot> #link https://drive.google.com/file/d/0B_YlsOGofvCCZFc3UnZ1MkxBNG8/view?usp=sharing
09:34:17 <acabot> #info We need more reviewers on patches
09:34:39 <acabot> today most of the reviews are done by core contributors
09:34:53 <acabot> and I think we need more +1 before each +2
09:35:02 <acabot> so please review some code
09:35:09 <vincentfrancoise> +1 on that
09:35:16 <acabot> licanwei : thank you for taking all these reviews today
09:35:44 <licanwei> :)
09:36:04 <acabot> #action sballe jwcroppe hvprash try to convince people to do more reviews on the project
09:36:42 <acabot> any other discussion to open ?
09:36:54 <diga_> acabot: is it ok if I start reviewing the code ?
09:37:14 <vincentfrancoise> diga_: anytime you want :)
09:37:15 <acabot> diga_ : of course you are more than welcome to review
09:37:32 <diga_> vincentfrancoise: acabot : thank you!
09:37:54 <acabot> diga_ : if you start reviewing, please reply with a comment
09:38:02 <diga_> acabot: sure
09:38:14 <acabot> every +1/-1 or even 0 is counted
09:38:21 <diga_> okay
09:38:48 <acabot> if you look at this #link http://stackalytics.com/?module=watcher-group
09:38:57 <diga_> acabot: this is good way to understand watcher completely
09:39:00 <diga_> sure
09:39:04 <acabot> 72% of reviews are done by b@#com
09:39:40 <acabot> we should be around 50%
09:40:10 <acabot> every new contributor reviewing code is good for the project
09:40:17 <diga_> Yes
09:40:45 <acabot> if you dont have any other topic to discuss, we can close the meeting
09:40:53 <acabot> thank you
09:41:03 <diga_> I appreciate b-com teams efforts on this project, I see it everydy
09:41:15 <acabot> diga_ : thx
09:41:28 <acabot> bye
09:41:31 <vincentfrancoise> bye
09:41:34 <diga_> acabot: wc! bye
09:41:34 <licanwei> bye
09:41:38 <acabot> #endmeeting