14:01:22 #startmeeting watcher 14:01:23 Meeting started Wed Jun 29 14:01:22 2016 UTC and is due to finish in 60 minutes. The chair is acabot. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:01:24 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:01:27 The meeting name has been set to 'watcher' 14:01:28 hi 14:01:31 o/ 14:01:35 hi 14:01:46 hi 14:01:52 hi 14:01:56 agenda for today #link https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#06.2F29.2F2016 14:01:56 hi 14:02:04 hi 14:02:04 hi 14:02:11 \o 14:02:14 o/ 14:02:16 hi 14:02:18 hi 14:02:34 #topic Announcements 14:03:00 One more person joined the meetup: Digambar Patil 14:03:15 welcome 14:03:29 welcome Digambar 14:03:34 +1 14:03:35 welcome 14:03:37 +1 14:03:38 welcome 14:03:56 working for Intel ? 14:04:00 no 14:04:06 welcome 14:04:12 I am assuming it is diga 14:04:31 diga: who are you working for? 14:04:33 thanks acabot 14:04:38 Persistent 14:04:46 cool! 14:04:51 sballe_: My name is Digambar 14:04:53 :) 14:05:07 thx for the intro 14:05:23 Thank you all 14:05:33 diga: Yes i know. I was refering to your IRC nickname 14:05:46 sballe_: yep :) 14:05:46 #info Watcher packages are available for Red Hat & Debian 14:06:11 Good news! 14:06:20 so danpawlik working on puppet packaging managed to have watcher packages for RDO & Debian (experimental) 14:06:41 where are the packages being hosted? 14:06:43 should we update our user doc to mention these packages ? 14:07:01 michaelgugino : https://packages.debian.org/experimental/python-watcherclient for deban 14:07:27 acabot: I would say yes 14:07:45 we currently have a "Installing from Source" and a "Installing from packages: PyPI" 14:08:02 +1 14:08:05 ok lets create a a new section for it 14:08:10 so that would seem kind of logic to add a "Installing from packages: Debian" and so son 14:08:11 +1 14:08:21 s/son/on 14:08:31 #action vincentfrancoise add a section "installing from packages" in user guide 14:08:34 Do we need check debian/redhat doc for watcher? 14:09:04 gzhai2 : what do you mean ? 14:09:23 debian/redhat should have their own openstack doc. 14:09:32 gzhai2: yes it a good idea to check the doc with the packages 14:09:35 install from rpm, and how to config it. 14:10:10 anyone wants to test packages ? 14:10:13 what version of RDO and debian is Watcher included in? 14:10:30 acabot: I can test Debian ones 14:10:40 sballe_ : its not included...its available 14:10:53 0.27 14:11:10 #action dtardivel test Watcher debian packages and update the install doc if needed 14:11:17 0.26 sorry 14:11:21 I can try redhat 14:11:39 actually I since Debian has 0.27 & RDO 0.26 :-) 14:11:40 jed56: watcher : python-watcher (0.27.0-2) 14:11:53 ah yes :p 14:12:05 yes this is because RDO wants final release so 0.26 14:12:38 #info 2 weeks away from submission deadline, please register as speakers in , we also need to complete descriptions 14:12:47 #link https://etherpad.openstack.org/p/Watcher_abstracts_barcelona2016 14:12:58 acabot: I will work on the abstracts this week. 14:13:00 I would like to have 3 speakers for each presentation 14:13:06 +1 14:13:33 thx for reminding. will update it 14:13:36 #action sballe_ work on the abstracts for Barcelona summit 14:13:37 https://etherpad.openstack.org/p/Watcher_abstracts_barcelona2016 14:13:44 sballe_ : thx 14:14:24 dont forget that if the presentation is accepted, every speaker will get a free pass ;-) 14:14:50 #info a first draft of mid-cycle agenda is available 14:14:57 #link https://etherpad.openstack.org/p/newton-watcher-midcycle 14:15:14 feel free to add topics you would like to discuss 14:15:18 acabot: Did Watcher contributors not get ATC passes? 14:15:30 there is still a lot of space in the agenda 14:15:35 edleafe : actually yes 14:15:45 edleafe : some of them at least 14:16:02 edleafe: yes 14:16:09 we have 14:16:10 OK, good. That's one of the benefits of being in the big tent :) 14:16:20 edleafe: +1 14:16:33 +1 14:16:39 edleafe : yes it is but to my understanting, it was related to the previous cycle... 14:17:17 Previous, and current up to the date they send out the ATC codes 14:17:28 edleafe : ok 14:17:37 #info Watcher should respect the Nova policies spec has been discussed with the Nova scheduler team 14:17:38 Since Watcher was in before then, I figured you should be good 14:17:50 #link https://review.openstack.org/#/c/329873/ 14:18:04 edleafe : thx Ed for adding it to the agenda 14:18:16 #link http://eavesdrop.openstack.org/meetings/nova_scheduler/2016/nova_scheduler.2016-06-27-14.00.log.html 14:18:52 I have already received ATC from other projects, fyi 14:18:58 as next steps, bauzas suggests that we drop a line about watcher in Nova mid-cycle agenda #link https://etherpad.openstack.org/p/newton-nova-scheduler 14:18:59 for this cycle, I mean. 14:19:10 acabot: +1 14:19:35 acabot: yes, this will certainly be discussed at the midcycle 14:19:39 and we submit a backlog spec #link http://specs.openstack.org/openstack/nova-specs/specs/backlog/ 14:19:46 to explain our problem 14:20:01 No schedule is set yet, but when it is, it would probably be a good time to come join us 14:20:08 since we'll be in the same building 14:20:10 :) 14:20:27 edleafe : do you think having a backlog spec on Nova side is a good way to progress ? 14:20:51 Sure - it will give people some background 14:20:57 ok 14:21:05 ok 14:21:25 #action jed56 acabot submit a backlog spec for Nova regarding policies management in Watcher 14:21:42 any other announcement ? 14:21:59 #topic Review Action Items 14:22:10 Update the continuously optimization spec needs reviews from core as its already implemented 14:22:19 #link https://review.openstack.org/#/c/330021/ 14:22:36 sballe_ : can you look at it and W+1 ? 14:22:43 yes 14:22:53 #action sballe_ review https://review.openstack.org/#/c/330021/ 14:23:02 Cluster model objects wrapper needs reviews from core 14:23:08 #link https://review.openstack.org/#/c/287019/ 14:23:21 ok add me to this one too 14:23:30 #action sballe_ jwcroppe review https://review.openstack.org/#/c/287019/ 14:23:50 Graph model describing virtual and physical elements in a data center & Consolidation of multiple Action-plans into a single one need reviews 14:23:57 #link https://review.openstack.org/#/c/298891/ 14:24:02 #link https://review.openstack.org/#/c/298871/ 14:24:22 sballe_ : we have to discuss about the implementation of these specs from Intel 14:24:33 I'll review those too since I am preparing a talk about Intel's contribution to Watcher 14:24:43 sballe_ : I will add it to midcycle agenda 14:25:04 make sense. the person implementing them will be at the mid-cycle 14:25:08 #action sballe_ review https://review.openstack.org/#/c/298891/ & https://review.openstack.org/#/c/298871/ 14:25:12 and so will Thijs 14:25:22 sballe_ : great ;-) 14:25:29 I'm starting to review 298871 14:25:32 moving to Watcher 14:25:53 #action jed56 review https://review.openstack.org/#/c/298871/ 14:26:00 Enable strategy parameters is in progress, test feedback requested 14:26:07 #link https://review.openstack.org/#/c/297556/ 14:26:27 vincentfrancoise : I think you are still testing it dont you ? 14:26:57 acabot: actually it was more dtardivel who was doing the testing on this 14:27:01 acabot: I tested it on devstack as well 14:27:14 dtardivel: any issue? 14:27:40 no, except data output format on console. Please see the last review 14:28:07 #action gzhai2 add a new PS to https://review.openstack.org/#/c/297556/ 14:28:16 Add continuously optimization needs reviews from core 14:28:23 #link https://review.openstack.org/#/c/321411/ 14:28:44 I this this has been reviewed and so can be merged asap 14:28:52 s/this/think 14:29:07 along with the spec :-D 14:29:34 Add scoring engines to database and API layers needs reviews 14:29:35 acabot: IMHO, we will be able to merged it soon 14:29:46 #link https://review.openstack.org/#/c/330382/ 14:30:11 you can add me 14:30:15 I did some reviews on this one 14:30:16 tkaczynski : you just submited a new PS, still waiting for reviews I suppose 14:30:35 #action jed56 review https://review.openstack.org/#/c/330382/ 14:30:47 add me 2 14:30:56 #action sballe_ review https://review.openstack.org/#/c/330382/ 14:31:07 I think I need to add some tests before it can be made +2 14:31:22 tkaczynski : ok 14:31:37 do you know your current coverage ? 14:31:57 I will try to do this this week, so we can close that. Vincent sent me the coverage. can I get that locally too? 14:32:09 tkaczynski: yes 14:32:10 tkaczynski: tox -e cover 14:32:17 great, thanks 14:32:30 #action tkaczynski add tests for https://review.openstack.org/#/c/330382/ 14:32:40 Add policies for API access control to watcher project needs reviews 14:32:48 #link https://review.openstack.org/#/c/334163/ 14:32:49 we should add a new rule in the gate each new patchset have to improve the code coverage :p 14:33:01 +1 14:33:16 hanrong1 : thx for this 1st contribution to watcher ! 14:33:57 hanrong1 : you also need to add tests ;-) 14:34:10 jed56: I tried to find some coverage ratcheting jobs but there was none 14:34:38 Add rally-jobs folder to get rally support needs new tests 14:34:38 vincentfrancoise: okay :) 14:34:45 #link https://review.openstack.org/#/c/333371/ 14:34:55 a lot of tests missing today ! 14:35:00 acabot: yes I asked to hanrong1 to push a new pachset on watcher-dasboard which is able to handle watcher policies 14:35:09 acabot: Thank you for all help 14:35:34 #action hanrong1 submit a new PS on watcher-dashboard to handle policies 14:35:36 thanks guys for my reviews too! :) 14:36:00 hanrong1: I will submit a new patch set for tests. 14:36:41 #action hanrong1 submit a new patch set for tests on https://review.openstack.org/#/c/334163/ 14:36:51 Enable strategy parameter needs a new PS 14:36:57 #link https://review.openstack.org/#/c/297557 14:37:27 #action gzhai2 submit a new PS for https://review.openstack.org/#/c/297557 14:37:40 Add scoring engine commands needs reviews 14:37:47 #link https://review.openstack.org/#/c/330385/ 14:37:59 #action jed56 sballe_ review https://review.openstack.org/#/c/330385/ 14:38:08 same reviewers for the CLI ;-) 14:38:29 Add support continuously-optimization needs a new PS 14:38:36 #link https://review.openstack.org/#/c/327095/ 14:39:04 anyone know the IRC nick for Vladimir ? 14:39:35 dtardivel : can you review https://review.openstack.org/#/c/327095/ as you will be working on the dashboard ? 14:40:01 acabot: yes of course 14:40:10 acabot: he's not connected but that was something like ostroverkhov 14:40:29 #action dtardivel review https://review.openstack.org/#/c/327095/ 14:40:48 vincentfrancoise : an easy one ;-) 14:41:50 I forgot to mention WIP around https://review.openstack.org/#/q/topic:bp/persistent-audit-parameters 14:42:17 hvprash michaelgugino : are you waiting for reviews on your code ? 14:42:18 acabot, yeah its in progress. hoping for review next week :) 14:42:25 we are not yet done 14:42:29 hvprash : ok thx 14:42:34 no, I'm not. There are some good reviews already, I am going to work on the patch this week 14:42:41 great 14:42:51 #topic Blueprint/Bug Review and Discussion 14:43:19 #info our current milestone is Newton-2 (due July 11th) 14:43:27 #link https://launchpad.net/watcher/+milestone/newton-2 14:43:57 as we are 2 weeks away from the release, we need to focus on code reviewing and bug fixes 14:44:34 we should probably postpone to Newton-3 https://blueprints.launchpad.net/watcher/+spec/persistent-audit-parameters 14:44:48 and https://blueprints.launchpad.net/watcher/+spec/cluster-model-objects-wrapper 14:45:11 tkaczynski : what about https://blueprints.launchpad.net/watcher/+spec/scoring-module ? 14:45:44 hvprash : what do you think about your BP ? 14:45:59 I need to add more details on our use case 14:46:07 then we could discuss 14:46:12 acabot: does that mean we shouldn't accept any pending changeset from these BPs? 14:46:49 because the 1st part of my BP could be merged before then I believe 14:46:51 vincentfrancoise : no the idea is to focus our effort on completing BPs 14:46:52 acabot, assuming you are asking about this https://blueprints.launchpad.net/watcher/+spec/workload-prioritization-qos 14:46:57 acabot: I'm working on it and making progress. will have code for review soon, maybe next week? 14:47:30 hvprash : this one will be discussed during the mid-cycle as its a 'large-scope" BP 14:47:37 yes 14:48:17 tkaczynski : ok as the freeze will be end of next week, I dont think we will be able to have it in this release 14:48:45 acabot: strictly speaking probably not, but soon after. is that okay? 14:49:09 vincentfrancoise : we will merge Part 1 if there is no dependency with parts 2 & 3 14:49:33 tkaczynski : in that case it will be Newton-3, I'm fine with that 14:49:48 but we can merge first 2 changesets 14:49:54 acabot: no there's none, that's why I already pushed the doc regarding this part 14:49:57 acabot: there is no dependency 14:50:36 good to know, in this case we will merge vincentfrancoise & tkaczynski current code 14:50:55 10 minutes left for open discussions 14:51:02 #topic Open Discussion 14:51:14 Global Requirement: is watcher-dashboard project automatically sync with OS Global Requirements ? 14:51:14 acabot: I am setting up watcher. I am going through the doc it seems you have lot of planned things. I would like to contribute to this project as I have some ideas also but I will start with adding bug fixes or whichever you think better place for me to start 14:51:32 #undo 14:51:33 Removing item from minutes: 14:51:59 I forgot to mention that we have bugs open in triaged state 14:52:09 okay 14:52:16 diga: I am also setting up watcher in my own lab s 14:52:30 diga : please have a look at https://launchpad.net/watcher/+milestone/newton-2 14:52:44 dtardivel: Is adding policies to watcher-dasboard a part of BP watcher-policies 14:52:46 we should have low-hanging-fruit 14:53:04 acabot: we should sync watcher-dashboard, as discusses with vincentfrancoise offline. 14:53:06 hanrong1: yes 14:53:06 acabot: that should be good starting place 14:53:20 diga : https://bugs.launchpad.net/watcher/+bugs?field.tag=low-hanging-fruit 14:53:48 sballe_: I am setting it with devstack but it seems watche many configuration params 14:53:59 dtardivel : are you still talking about policies ? 14:54:01 diga: yeah same here 14:54:20 acabot: thanks I will pick some 14:54:21 acabot: sorry. sync with globla requirements 14:54:36 ok let me move back to open discussions 14:54:45 #topic Open discussion 14:54:58 Global Requirement: is watcher-dashboard project automatically sync with OS Global Requirements ? 14:55:29 acabot: yes it should be :) 14:55:37 dtardivel vincentfrancoise : ok so you agreed that we should do it 14:55:42 any comment on this ? 14:55:58 dtardivel: +1 14:56:12 #action dtardivel add OS global requirements to watcher-dashboard 14:56:19 How to deal with packaging (puppet) when updating conf parameters ? 14:56:32 we probably need danpawlik on this 14:57:34 the problem is that we need to maintain puppet recipes now 14:57:45 and Watcher parameters are still evolving 14:58:25 acabot: IMHO,we have to carefully write the "deployer impact" section in the specs 14:58:52 jed56 : ok 14:59:19 acabot: how to notify the pupper maintainers ? 14:59:29 we will probably discuss it again in the rc-final version as packaging will be tested 14:59:38 s/pupper/puppet 14:59:50 do we need to create bugs ? 14:59:51 jed56: acabot there is no specific tag in commit message related to packaging 15:00:05 dtardivel: ah :( 15:00:22 #action acabot ask packaging team how to deal with parameters 15:00:27 time to end 15:00:30 thx 15:00:37 #endmeeting