| *** harlowja has joined #openstack-watcher | 00:06 | |
| *** danpawli1 has joined #openstack-watcher | 00:17 | |
| *** danpawlik has quit IRC | 00:20 | |
| *** thorst has joined #openstack-watcher | 00:33 | |
| *** thorst has quit IRC | 00:34 | |
| *** lamt has joined #openstack-watcher | 00:39 | |
| *** jwcroppe has quit IRC | 00:41 | |
| *** zhurong has joined #openstack-watcher | 00:56 | |
| *** jwcroppe has joined #openstack-watcher | 00:56 | |
| *** zhurong has quit IRC | 00:56 | |
| *** zhurong has joined #openstack-watcher | 00:56 | |
| *** jwcroppe has quit IRC | 00:56 | |
| *** jwcroppe has joined #openstack-watcher | 00:56 | |
| *** hidekazu has joined #openstack-watcher | 01:08 | |
| *** lamt has quit IRC | 01:22 | |
| *** thorst has joined #openstack-watcher | 01:35 | |
| *** thorst has quit IRC | 01:40 | |
| *** diablo_rojo has joined #openstack-watcher | 01:48 | |
| *** diablo_rojo has quit IRC | 01:48 | |
| *** diablo_rojo has joined #openstack-watcher | 01:49 | |
| diablo_rojo | Hello, just wanted to give you all a reminder that the period for the Travel Support Program for the PTG ends today. We would love to get more people from watcher to the PTG. Here is the link to the TSP Application: https://openstackfoundation.formstack.com/forms/travelsupportptg_atlanta | 01:49 |
|---|---|---|
| *** thorst has joined #openstack-watcher | 01:50 | |
| *** thorst has quit IRC | 01:50 | |
| *** diga has quit IRC | 01:54 | |
| openstackgerrit | Cao Xuan Hoang proposed openstack/watcher-dashboard: Replaces yaml.load() with yaml.safe_load() https://review.openstack.org/421624 | 02:14 |
| *** thorst has joined #openstack-watcher | 02:30 | |
| *** thorst has quit IRC | 02:30 | |
| *** harlowja has quit IRC | 02:36 | |
| *** thorst has joined #openstack-watcher | 02:56 | |
| *** thorst has quit IRC | 02:56 | |
| *** zhurong has quit IRC | 03:00 | |
| *** zhurong has joined #openstack-watcher | 03:02 | |
| *** zhurong has quit IRC | 03:23 | |
| *** thorst has joined #openstack-watcher | 03:27 | |
| *** diablo_rojo has quit IRC | 03:27 | |
| *** thorst has quit IRC | 03:33 | |
| *** diga has joined #openstack-watcher | 04:40 | |
| *** jwcroppe has quit IRC | 04:45 | |
| *** hidekazu has quit IRC | 04:59 | |
| *** thorst has joined #openstack-watcher | 05:29 | |
| *** thorst has quit IRC | 05:34 | |
| *** adisky_ has joined #openstack-watcher | 05:50 | |
| openstackgerrit | Rong Han proposed openstack/watcher-specs: Support Description For Dynamic Action https://review.openstack.org/401111 | 07:09 |
| *** thorst has joined #openstack-watcher | 07:30 | |
| *** thorst has quit IRC | 07:35 | |
| *** zhurong has joined #openstack-watcher | 07:37 | |
| *** zhurong has quit IRC | 07:46 | |
| *** thorst has joined #openstack-watcher | 08:00 | |
| *** thorst has quit IRC | 08:00 | |
| *** pksingh has joined #openstack-watcher | 08:10 | |
| pksingh | Hello Guys, is there a meeting today? | 08:13 |
| *** alexchadin has joined #openstack-watcher | 08:15 | |
| *** tmetsch has joined #openstack-watcher | 08:27 | |
| *** vincentfrancoise has joined #openstack-watcher | 08:32 | |
| pksingh | hello vincentfrancoise | 08:39 |
| openstackgerrit | Alexander Chadin proposed openstack/watcher: New Applier Workflow Engine https://review.openstack.org/421297 | 08:40 |
| *** alexchadin has quit IRC | 08:44 | |
| *** alexchadin has joined #openstack-watcher | 08:45 | |
| pksingh | alexchadin: is there watcher team meeting today? | 08:53 |
| alexchadin | hi pksingh | 08:53 |
| pksingh | alexchadin: hello | 08:53 |
| alexchadin | pksingh: There will be meeting today | 08:53 |
| alexchadin | in 7 minutes | 08:53 |
| pksingh | alexchadin: ok great, thanks, will attend | 08:54 |
| alexchadin | pksingh: you are welcome | 08:54 |
| pksingh | alexchadin: its on openstack-meeting-4 right? just for confirmation | 08:54 |
| alexchadin | pksingh: right | 08:54 |
| pksingh | alexchadin: k, thnx | 08:55 |
| *** dtardivel has joined #openstack-watcher | 09:01 | |
| acabot | meeting starts on #openstack-meeting-4 | 09:01 |
| vincentfrancoise | alexchadin: for the new planner, I need to check it in-depth | 09:03 |
| vincentfrancoise | alexchadin: I'll do it today | 09:03 |
| alexchadin | vincentfrancoise: I need to complete spec for planner by adding some info about weight planner | 09:04 |
| vincentfrancoise | alexchadin: I was talking about the code part so that's fine | 09:05 |
| alexchadin | vincentfrancoise: yeah, I know, just wanted you to know my priority today:) feel free to ask | 09:06 |
| vincentfrancoise | alexchadin: thx for letting me know, I'll review your specs today as well if I can | 09:06 |
| *** hidekazu has joined #openstack-watcher | 09:20 | |
| openstackgerrit | Alexander Chadin proposed openstack/watcher: New Applier Workflow Engine https://review.openstack.org/421297 | 09:27 |
| openstackgerrit | David TARDIVEL proposed openstack/watcher: Remove blackslashes in Watcher codebase https://review.openstack.org/412484 | 09:42 |
| *** alexchadin has quit IRC | 09:58 | |
| *** alexchadin has joined #openstack-watcher | 10:01 | |
| *** thorst has joined #openstack-watcher | 10:01 | |
| openstackgerrit | Alexander Chadin proposed openstack/watcher: New Applier Workflow Engine https://review.openstack.org/421297 | 10:03 |
| openstackgerrit | Merged openstack/watcher: Should use glanceclient to get images https://review.openstack.org/414445 | 10:05 |
| *** thorst has quit IRC | 10:06 | |
| *** hidekazu has left #openstack-watcher | 10:08 | |
| *** alexchadin has quit IRC | 10:08 | |
| *** alexchadin has joined #openstack-watcher | 10:08 | |
| *** alexchadin has quit IRC | 10:08 | |
| *** alexchadin has joined #openstack-watcher | 10:09 | |
| *** alexchadin has quit IRC | 10:09 | |
| *** alexchadin has joined #openstack-watcher | 10:10 | |
| *** alexchadin has quit IRC | 10:10 | |
| *** atuly_ has quit IRC | 10:18 | |
| *** atuly_ has joined #openstack-watcher | 10:30 | |
| *** vincentfrancoise has quit IRC | 11:02 | |
| *** alexchadin has joined #openstack-watcher | 11:10 | |
| *** pksingh has quit IRC | 11:23 | |
| openstackgerrit | Alexander Chadin proposed openstack/watcher: New Applier Workflow Engine https://review.openstack.org/421297 | 11:31 |
| *** acabot has quit IRC | 11:40 | |
| *** alexchadin has quit IRC | 11:40 | |
| *** alexchadin has joined #openstack-watcher | 11:41 | |
| *** thorst has joined #openstack-watcher | 12:02 | |
| *** thorst has quit IRC | 12:06 | |
| *** acabot has joined #openstack-watcher | 12:21 | |
| *** thorst has joined #openstack-watcher | 12:51 | |
| *** zhurong has joined #openstack-watcher | 12:53 | |
| *** vincentfrancoise has joined #openstack-watcher | 13:10 | |
| *** zhurong has quit IRC | 13:11 | |
| *** zhurong has joined #openstack-watcher | 13:12 | |
| *** zhurong has quit IRC | 13:13 | |
| *** lamt has joined #openstack-watcher | 13:23 | |
| *** lamt has quit IRC | 13:24 | |
| openstackgerrit | Alexander Chadin proposed openstack/watcher-specs: Add improvements to the planner and workflow mechanisms https://review.openstack.org/385871 | 13:31 |
| *** zhurong has joined #openstack-watcher | 13:32 | |
| *** diga has quit IRC | 13:32 | |
| *** alexchadin has quit IRC | 13:43 | |
| *** alexchadin has joined #openstack-watcher | 13:47 | |
| *** jwcroppe has joined #openstack-watcher | 14:03 | |
| dtardivel | alexchadin: hi Alex | 14:45 |
| dtardivel | alexchadin: I have issues your new planner changeset. I think you forgot to report fixes on weight.py pushed on https://review.openstack.org/#/c/415505/2/watcher/decision_engine/planner/weight.py | 14:47 |
| dtardivel | alexchadin: sorry it's my fault :/ I forgot to push remview on https://review.openstack.org/#/c/415505/2 | 14:50 |
| dtardivel | So the file weight.py on 'New default planner' has issues I reported on https://review.openstack.org/#/c/415505/2/watcher/decision_engine/planner/weight.py | 14:51 |
| *** alexchadin has quit IRC | 14:53 | |
| sballe_ | mornign | 14:54 |
| *** alexchadin has joined #openstack-watcher | 14:55 | |
| alexchadin | hi dtardivel | 14:55 |
| acabot | sballe_ : morning | 14:55 |
| sballe_ | sorry I missed the iRC meeting but I am in California | 14:55 |
| alexchadin | dtardivel: what was your latest message? | 14:55 |
| acabot | sballe_ : ttx told me this morning that PTG is almost full | 14:55 |
| acabot | sballe_ : 100 seats left | 14:55 |
| dtardivel | alexchadin: Are you again at work office ? | 14:55 |
| sballe_ | yeah I know chrisspencer and pshedimb are going | 14:55 |
| acabot | sballe_ : so I registered with David | 14:55 |
| alexchadin | dtardivel: yeap | 14:56 |
| acabot | sballe_ : did they register already ? | 14:56 |
| sballe_ | I have a leadership meeting so I cannot attend. Same weel :-( | 14:56 |
| sballe_ | acabot: yes | 14:56 |
| sballe_ | I believe Thijs will go too. tmetsch | 14:56 |
| acabot | ok and did they mention that they will attend watcher meetings ? | 14:56 |
| sballe_ | yes I believe so given taht taht is their main priotity | 14:56 |
| acabot | ok | 14:57 |
| sballe_ | chrisspencer: pshedimb ? | 14:57 |
| vincentfrancoise | alexchadin: I also found an issue on the planner and I am trying to write the unit test that covers it before fixing it | 14:57 |
| alexchadin | vincentfrancoise: what issue have you found? | 14:58 |
| dtardivel | alexchadin: Can you push on update on your changeset 'New default planner' now ? if not, I will do it, because I already provided the fix on POC version pushed by vincentfrancoise. | 14:58 |
| vincentfrancoise | alexchadin: I also changed uuids in the test code for the weight planner because it was really difficult to read (now it's 11111111-1111-1111-1111-111111111111, ...) | 14:59 |
| chrisspencer | sballe_ we registered already but have not gotten the final ok to book flights and hotel. Pending budget approval | 14:59 |
| alexchadin | dtardivel: let me see | 14:59 |
| vincentfrancoise | alexchadin: basically it creates a loop from the last action back to the first one somehow | 14:59 |
| sballe_ | chrisspencer: I would ask Mike if you can book the hotel before they run out of rooms in convenient hotels. Just make sure you can cancel the hotel | 15:00 |
| sballe_ | pshedimb: should do the same | 15:01 |
| pshedimb | sballe_, I spoke to Mike yesterday and he asked to book the hotel | 15:01 |
| alexchadin | dtardivel: does self.config.parallelization really return str? | 15:01 |
| sballe_ | perfect :) | 15:01 |
| pshedimb | sballe_, Though he asked us to wait to book the air tickets | 15:01 |
| sballe_ | makes total sense. | 15:02 |
| alexchadin | dtardivel: ping | 15:05 |
| sballe_ | acabot: What was decided ont he talks for the summit? | 15:06 |
| sballe_ | I talked to Mrittika and we need to have another chat on the finerprinting talk | 15:06 |
| sballe_ | acabot: Are you there? | 15:08 |
| vincentfrancoise | sballe_: he's not at his desk but he'll probably be back soon | 15:14 |
| *** openstack has joined #openstack-watcher | 15:17 | |
| *** pshedimb has quit IRC | 15:19 | |
| *** Daviey has quit IRC | 15:19 | |
| *** pshedimb has joined #openstack-watcher | 15:21 | |
| *** Daviey has joined #openstack-watcher | 15:22 | |
| alexchadin | dtardivel: where did you import types module from? | 15:23 |
| *** alexchadin has quit IRC | 15:23 | |
| *** alexchadin has joined #openstack-watcher | 15:23 | |
| vincentfrancoise | from oslo_config import types | 15:24 |
| vincentfrancoise | alexchadin: ^ | 15:24 |
| sballe_ | ok thx | 15:24 |
| alexchadin | vincentfrancoise: thx | 15:25 |
| *** Tamayo has joined #openstack-watcher | 15:25 | |
| *** Tamayo has quit IRC | 15:30 | |
| *** Tamayo has joined #openstack-watcher | 15:32 | |
| alexchadin | vincentfrancoise: my tests fail if I add `type` argument | 15:58 |
| vincentfrancoise | alexchadin: what kind of failure? | 15:58 |
| alexchadin | http://pastebin.com/HGhRSNB6 | 15:59 |
| alexchadin | vincentfrancoise: ^ | 15:59 |
| vincentfrancoise | alexchadin: is your field an cfg.Opt one? | 16:04 |
| vincentfrancoise | instead of a DictOpt I mean | 16:04 |
| alexchadin | vincentfrancoise: thanks for notice | 16:05 |
| alexchadin | vincentfrancoise: trying again | 16:05 |
| vincentfrancoise | alexchadin: ok :p | 16:05 |
| vincentfrancoise | alexchadin: ok so there's no bug in what you did actually | 16:11 |
| vincentfrancoise | alexchadin: but you added more edges than I did | 16:12 |
| alexchadin | vincentfrancoise: you looked through my planner? | 16:12 |
| vincentfrancoise | alexchadin: which is better than what I did in the 1st place | 16:13 |
| vincentfrancoise | alexchadin: no just the weight planner for now | 16:13 |
| vincentfrancoise | alexchadin: also, I added a new unit test with more actions to make sure it works | 16:13 |
| alexchadin | vincentfrancoise: hm. what edges you speak about?:) | 16:13 |
| vincentfrancoise | alexchadin: https://review.openstack.org/#/c/415505/2/watcher/decision_engine/planner/weight.py@121 | 16:14 |
| vincentfrancoise | alexchadin: this is actually not the reality | 16:14 |
| vincentfrancoise | alexchadin: what you do is http://paste.openstack.org/show/595420/ | 16:15 |
| vincentfrancoise | alexchadin: which is more accurate if only want 2 migrate running at a time | 16:15 |
| alexchadin | vincentfrancoise: oh, I see | 16:15 |
| vincentfrancoise | alexchadin: that's why I thought there was a bug | 16:16 |
| alexchadin | vincentfrancoise: It makes more sense to me | 16:16 |
| alexchadin | vincentfrancoise: but, why do we need action_graph? :D | 16:17 |
| vincentfrancoise | alexchadin: can I send you my test_weight planner over so you push it yourself? | 16:17 |
| alexchadin | vincentfrancoise: I just noticed that we don't use it anywhere | 16:18 |
| alexchadin | vincentfrancoise: of course | 16:18 |
| vincentfrancoise | alexchadin: here is the test_weight_planner.py http://paste.openstack.org/show/595422/ | 16:19 |
| vincentfrancoise | alexchadin: didn't check any pep8 on it BTW | 16:20 |
| vincentfrancoise | for action_graph... | 16:20 |
| vincentfrancoise | alexchadin: we don't need it anymore now | 16:20 |
| alexchadin | vincentfrancoise: then I would like to remove it | 16:21 |
| vincentfrancoise | alexchadin: ok go for it | 16:21 |
| vincentfrancoise | alexchadin: you can also remove the networkx requirements in that case | 16:22 |
| alexchadin | vincentfrancoise: | 16:22 |
| vincentfrancoise | alexchadin: now I review the other planner :p | 16:23 |
| alexchadin | vincentfrancoise: oh, it is used | 16:23 |
| alexchadin | vincentfrancoise: action_graph is used by some methods | 16:23 |
| alexchadin | vincentfrancoise: I will take a look at these methods tomorrow | 16:23 |
| vincentfrancoise | alexchadin: I don't care about merging it as-is to be honest | 16:24 |
| alexchadin | vincentfrancoise: me too | 16:24 |
| alexchadin | vincentfrancoise: don't want to get some more bugs now:) | 16:24 |
| vincentfrancoise | alexchadin: it works and it doesn't add too much overhead | 16:24 |
| vincentfrancoise | alexchadin: and I still have the 2nd planner to review -_- | 16:25 |
| alexchadin | vincentfrancoise: workload stabilization? have a good luck! :D | 16:25 |
| vincentfrancoise | alexchadin: I might not finish it today though since I took like 2h to review the 1st one | 16:26 |
| vincentfrancoise | alexchadin: one last thing | 16:27 |
| vincentfrancoise | alexchadin: turn_host_to_acpi_s3_state doesn't exist yet | 16:27 |
| vincentfrancoise | alexchadin: when do you plan on pushing it? it's because you mention it in your worload_stabilization planner | 16:28 |
| alexchadin | vincentfrancoise: I think it requires some discussions | 16:28 |
| alexchadin | vincentfrancoise: sorry, but I have to go | 16:29 |
| vincentfrancoise | alexchadin: ok :) good evening | 16:30 |
| alexchadin | vincentfrancoise: have a nice evening! | 16:30 |
| *** alexchadin has quit IRC | 16:31 | |
| chrisspencer | vincentfrancoise: I had a question about adding extra details to logs for bug: https://bugs.launchpad.net/watcher/+bug/1642623. How I can/should go about verifying my fix? | 16:33 |
| openstack | Launchpad bug 1642623 in watcher "Add resource ID in nova notification logs " [Wishlist,Triaged] - Assigned to Chris Spencer (christopher-m-spencer) | 16:33 |
| vincentfrancoise | chrisspencer: how can I help? | 16:34 |
| vincentfrancoise | chrisspencer: did you do a unit test to see if it get passed the message ID? | 16:34 |
| chrisspencer | vincentfrancoise: I ran the existing unit tests but haven't written a new one | 16:35 |
| vincentfrancoise | chrisspencer: in TestReceiveNovaNotifications (watcher/tests/decision_engine/model/notification/test_nova_notifications.py) | 16:37 |
| vincentfrancoise | chrisspencer: I would add a new test that mocks the logger to see if the log.info() gets called with the right parameters and what not | 16:37 |
| chrisspencer | vincentfrancoise: ack. will do | 16:38 |
| vincentfrancoise | chrisspencer: 1 test will probably do | 16:38 |
| *** harlowja has joined #openstack-watcher | 16:39 | |
| acabot | sballe_ : I'm back | 16:40 |
| acabot | sballe_ : we didn't talk much about Boston talks this morning (we were only 5 at the meeting) | 16:46 |
| sballe_ | acabot: ok thx. | 16:46 |
| acabot | sballe_ : fell free to suggest any new talk | 16:46 |
| sballe_ | I would like to move the nisy neighbor bp to Pike because we have a dependency on a collectd plugin that is not opensource yet, should be within the next couple of weeks | 16:47 |
| acabot | ok will do | 16:47 |
| sballe_ | acabot: I was going to submit the one on NN | 16:47 |
| openstackgerrit | David TARDIVEL proposed openstack/watcher: New Applier Workflow Engine https://review.openstack.org/421297 | 16:47 |
| sballe_ | hvprash: will be a co-speaker on taht one | 16:47 |
| acabot | sballe_ : great | 16:48 |
| acabot | sballe_ : http://eavesdrop.openstack.org/meetings/watcher/2017/watcher.2017-01-18-09.01.txt | 16:49 |
| acabot | I also opened the etherpad for PTG https://etherpad.openstack.org/p/pike-watcher-ptg | 16:49 |
| sballe_ | cool! | 16:49 |
| sballe_ | tmetsch: should be there as well. | 16:50 |
| acabot | cool | 16:50 |
| *** harlowja has quit IRC | 17:33 | |
| *** vincentfrancoise has quit IRC | 17:34 | |
| *** tmetsch has quit IRC | 18:02 | |
| *** absubram has joined #openstack-watcher | 18:35 | |
| *** absubram has quit IRC | 19:17 | |
| *** adisky_ has quit IRC | 19:19 | |
| *** absubram has joined #openstack-watcher | 19:22 | |
| *** dtardivel has quit IRC | 20:24 | |
| *** wootehfoot has joined #openstack-watcher | 20:33 | |
| *** absubram has quit IRC | 20:43 | |
| *** absubram has joined #openstack-watcher | 21:00 | |
| *** absubram has quit IRC | 21:21 | |
| *** absubram has joined #openstack-watcher | 21:33 | |
| *** thorst has quit IRC | 22:07 | |
| *** thorst has joined #openstack-watcher | 22:39 | |
| *** thorst has quit IRC | 22:41 | |
| openstackgerrit | Chris Spencer proposed openstack/watcher: Adding additional details to notification logs https://review.openstack.org/422295 | 22:45 |
| *** absubram has quit IRC | 23:09 | |
| *** pshedimb has quit IRC | 23:24 | |
| *** acabot has quit IRC | 23:24 | |
| *** atuly_ has quit IRC | 23:24 | |
| *** pshedimb has joined #openstack-watcher | 23:24 | |
| *** acabot has joined #openstack-watcher | 23:24 | |
| *** atuly_ has joined #openstack-watcher | 23:24 | |
| *** wootehfoot has quit IRC | 23:28 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!