Wednesday, 2017-01-18

*** harlowja has joined #openstack-watcher00:06
*** danpawli1 has joined #openstack-watcher00:17
*** danpawlik has quit IRC00:20
*** thorst has joined #openstack-watcher00:33
*** thorst has quit IRC00:34
*** lamt has joined #openstack-watcher00:39
*** jwcroppe has quit IRC00:41
*** zhurong has joined #openstack-watcher00:56
*** jwcroppe has joined #openstack-watcher00:56
*** zhurong has quit IRC00:56
*** zhurong has joined #openstack-watcher00:56
*** jwcroppe has quit IRC00:56
*** jwcroppe has joined #openstack-watcher00:56
*** hidekazu has joined #openstack-watcher01:08
*** lamt has quit IRC01:22
*** thorst has joined #openstack-watcher01:35
*** thorst has quit IRC01:40
*** diablo_rojo has joined #openstack-watcher01:48
*** diablo_rojo has quit IRC01:48
*** diablo_rojo has joined #openstack-watcher01:49
diablo_rojoHello, 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:
*** thorst has joined #openstack-watcher01:50
*** thorst has quit IRC01:50
*** diga has quit IRC01:54
openstackgerritCao Xuan Hoang proposed openstack/watcher-dashboard: Replaces yaml.load() with yaml.safe_load()
*** thorst has joined #openstack-watcher02:30
*** thorst has quit IRC02:30
*** harlowja has quit IRC02:36
*** thorst has joined #openstack-watcher02:56
*** thorst has quit IRC02:56
*** zhurong has quit IRC03:00
*** zhurong has joined #openstack-watcher03:02
*** zhurong has quit IRC03:23
*** thorst has joined #openstack-watcher03:27
*** diablo_rojo has quit IRC03:27
*** thorst has quit IRC03:33
*** diga has joined #openstack-watcher04:40
*** jwcroppe has quit IRC04:45
*** hidekazu has quit IRC04:59
*** thorst has joined #openstack-watcher05:29
*** thorst has quit IRC05:34
*** adisky_ has joined #openstack-watcher05:50
openstackgerritRong Han proposed openstack/watcher-specs: Support Description For Dynamic Action
*** thorst has joined #openstack-watcher07:30
*** thorst has quit IRC07:35
*** zhurong has joined #openstack-watcher07:37
*** zhurong has quit IRC07:46
*** thorst has joined #openstack-watcher08:00
*** thorst has quit IRC08:00
*** pksingh has joined #openstack-watcher08:10
pksinghHello Guys, is there a meeting today?08:13
*** alexchadin has joined #openstack-watcher08:15
*** tmetsch has joined #openstack-watcher08:27
*** vincentfrancoise has joined #openstack-watcher08:32
pksinghhello vincentfrancoise08:39
openstackgerritAlexander Chadin proposed openstack/watcher: New Applier Workflow Engine
*** alexchadin has quit IRC08:44
*** alexchadin has joined #openstack-watcher08:45
pksinghalexchadin: is there watcher team  meeting today?08:53
alexchadinhi pksingh08:53
pksinghalexchadin: hello08:53
alexchadinpksingh: There will be meeting today08:53
alexchadinin 7 minutes08:53
pksinghalexchadin: ok great, thanks, will attend08:54
alexchadinpksingh: you are welcome08:54
pksinghalexchadin: its on openstack-meeting-4 right? just for confirmation08:54
alexchadinpksingh: right08:54
pksinghalexchadin: k, thnx08:55
*** dtardivel has joined #openstack-watcher09:01
acabotmeeting starts on #openstack-meeting-409:01
vincentfrancoisealexchadin: for the new planner, I need to check it in-depth09:03
vincentfrancoisealexchadin: I'll do it today09:03
alexchadinvincentfrancoise: I need to complete spec for planner by adding some info about weight planner09:04
vincentfrancoisealexchadin: I was talking about the code part so that's fine09:05
alexchadinvincentfrancoise: yeah, I know, just wanted you to know my priority today:) feel free to ask09:06
vincentfrancoisealexchadin: thx for letting me know, I'll review your specs today as well if I can09:06
*** hidekazu has joined #openstack-watcher09:20
openstackgerritAlexander Chadin proposed openstack/watcher: New Applier Workflow Engine
openstackgerritDavid TARDIVEL proposed openstack/watcher: Remove blackslashes in Watcher codebase
*** alexchadin has quit IRC09:58
*** alexchadin has joined #openstack-watcher10:01
*** thorst has joined #openstack-watcher10:01
openstackgerritAlexander Chadin proposed openstack/watcher: New Applier Workflow Engine
openstackgerritMerged openstack/watcher: Should use glanceclient to get images
*** thorst has quit IRC10:06
*** hidekazu has left #openstack-watcher10:08
*** alexchadin has quit IRC10:08
*** alexchadin has joined #openstack-watcher10:08
*** alexchadin has quit IRC10:08
*** alexchadin has joined #openstack-watcher10:09
*** alexchadin has quit IRC10:09
*** alexchadin has joined #openstack-watcher10:10
*** alexchadin has quit IRC10:10
*** atuly_ has quit IRC10:18
*** atuly_ has joined #openstack-watcher10:30
*** vincentfrancoise has quit IRC11:02
*** alexchadin has joined #openstack-watcher11:10
*** pksingh has quit IRC11:23
openstackgerritAlexander Chadin proposed openstack/watcher: New Applier Workflow Engine
*** acabot has quit IRC11:40
*** alexchadin has quit IRC11:40
*** alexchadin has joined #openstack-watcher11:41
*** thorst has joined #openstack-watcher12:02
*** thorst has quit IRC12:06
*** acabot has joined #openstack-watcher12:21
*** thorst has joined #openstack-watcher12:51
*** zhurong has joined #openstack-watcher12:53
*** vincentfrancoise has joined #openstack-watcher13:10
*** zhurong has quit IRC13:11
*** zhurong has joined #openstack-watcher13:12
*** zhurong has quit IRC13:13
*** lamt has joined #openstack-watcher13:23
*** lamt has quit IRC13:24
openstackgerritAlexander Chadin proposed openstack/watcher-specs: Add improvements to the planner and workflow mechanisms
*** zhurong has joined #openstack-watcher13:32
*** diga has quit IRC13:32
*** alexchadin has quit IRC13:43
*** alexchadin has joined #openstack-watcher13:47
*** jwcroppe has joined #openstack-watcher14:03
dtardivelalexchadin: hi Alex14:45
dtardivelalexchadin: I have issues your new planner changeset. I think you forgot to report fixes on pushed on
dtardivelalexchadin: sorry it's my fault :/ I forgot to push remview on
dtardivelSo the file on 'New default planner' has issues I reported on
*** alexchadin has quit IRC14:53
*** alexchadin has joined #openstack-watcher14:55
alexchadinhi dtardivel14:55
acabotsballe_ : morning14:55
sballe_sorry I missed the iRC meeting but I am in California14:55
alexchadindtardivel: what was your latest message?14:55
acabotsballe_ : ttx told me this morning that PTG is almost full14:55
acabotsballe_ : 100 seats left14:55
dtardivelalexchadin: Are you again at work office ?14:55
sballe_yeah I know chrisspencer and pshedimb  are going14:55
acabotsballe_ : so I registered with David14:55
alexchadindtardivel: yeap14:56
acabotsballe_ : did they register already ?14:56
sballe_I have a leadership meeting so I cannot attend. Same weel :-(14:56
sballe_acabot:  yes14:56
sballe_I believe Thijs will go too. tmetsch14:56
acabotok and did they mention that they will attend watcher meetings ?14:56
sballe_yes I believe so given taht taht is their main priotity14:56
sballe_chrisspencer: pshedimb  ?14:57
vincentfrancoisealexchadin: I also found an issue on the planner and I am trying to write the unit test that covers it before fixing it14:57
alexchadinvincentfrancoise: what issue have you found?14:58
dtardivelalexchadin: 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
vincentfrancoisealexchadin: 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
chrisspencersballe_ we registered already but have not gotten the final ok to book flights and hotel.  Pending budget approval14:59
alexchadindtardivel: let me see14:59
vincentfrancoisealexchadin: basically it creates a loop from the last action back to the first one somehow14: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 hotel15:00
sballe_pshedimb: should do the same15:01
pshedimbsballe_, I spoke to Mike yesterday and he asked to book the hotel15:01
alexchadindtardivel: does self.config.parallelization really return str?15:01
sballe_perfect :)15:01
pshedimbsballe_, Though he asked us to wait to book the air tickets15:01
sballe_makes total sense.15:02
alexchadindtardivel: ping15: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 talk15:06
sballe_acabot: Are you there?15:08
vincentfrancoisesballe_: he's not at his desk but he'll probably be back soon15:14
*** openstack has joined #openstack-watcher15:17
*** pshedimb has quit IRC15:19
*** Daviey has quit IRC15:19
*** pshedimb has joined #openstack-watcher15:21
*** Daviey has joined #openstack-watcher15:22
alexchadindtardivel: where did you import types module from?15:23
*** alexchadin has quit IRC15:23
*** alexchadin has joined #openstack-watcher15:23
vincentfrancoisefrom oslo_config import types15:24
vincentfrancoisealexchadin: ^15:24
sballe_ok thx15:24
alexchadinvincentfrancoise: thx15:25
*** Tamayo has joined #openstack-watcher15:25
*** Tamayo has quit IRC15:30
*** Tamayo has joined #openstack-watcher15:32
alexchadinvincentfrancoise: my tests fail if I add `type` argument15:58
vincentfrancoisealexchadin: what kind of failure?15:58
alexchadinvincentfrancoise: ^15:59
vincentfrancoisealexchadin: is your field an cfg.Opt one?16:04
vincentfrancoiseinstead of a DictOpt I mean16:04
alexchadinvincentfrancoise: thanks for notice16:05
alexchadinvincentfrancoise: trying again16:05
vincentfrancoisealexchadin: ok :p16:05
vincentfrancoisealexchadin: ok so there's no bug in what you did actually16:11
vincentfrancoisealexchadin: but you added more edges than I did16:12
alexchadinvincentfrancoise: you looked through my planner?16:12
vincentfrancoisealexchadin: which is better than what I did in the 1st place16:13
vincentfrancoisealexchadin: no just the weight planner for now16:13
vincentfrancoisealexchadin: also, I added a new unit test with more actions to make sure it works16:13
alexchadinvincentfrancoise: hm. what edges you speak about?:)16:13
vincentfrancoisealexchadin: this is actually not the reality16:14
vincentfrancoisealexchadin: what you do is
vincentfrancoisealexchadin: which is more accurate if only want 2 migrate running at a time16:15
alexchadinvincentfrancoise: oh, I see16:15
vincentfrancoisealexchadin: that's why I thought there was a bug16:16
alexchadinvincentfrancoise: It makes more sense to me16:16
alexchadinvincentfrancoise: but, why do we need action_graph? :D16:17
vincentfrancoisealexchadin: can I send you my test_weight planner over so you push it yourself?16:17
alexchadinvincentfrancoise: I just noticed that we don't use it anywhere16:18
alexchadinvincentfrancoise: of course16:18
vincentfrancoisealexchadin: here is the
vincentfrancoisealexchadin: didn't check any pep8 on it BTW16:20
vincentfrancoisefor action_graph...16:20
vincentfrancoisealexchadin: we don't need it anymore now16:20
alexchadinvincentfrancoise: then I would like to remove it16:21
vincentfrancoisealexchadin: ok go for it16:21
vincentfrancoisealexchadin: you can also remove the networkx requirements in that case16:22
vincentfrancoisealexchadin: now I review the other planner :p16:23
alexchadinvincentfrancoise: oh, it is used16:23
alexchadinvincentfrancoise: action_graph is used by some methods16:23
alexchadinvincentfrancoise: I will take a look at these methods tomorrow16:23
vincentfrancoisealexchadin: I don't care about merging it as-is to be honest16:24
alexchadinvincentfrancoise: me too16:24
alexchadinvincentfrancoise: don't want to get some more bugs now:)16:24
vincentfrancoisealexchadin: it works and it doesn't add too much overhead16:24
vincentfrancoisealexchadin: and I still have the 2nd planner to review -_-16:25
alexchadinvincentfrancoise: workload stabilization? have a good luck! :D16:25
vincentfrancoisealexchadin: I might not finish it today though since I took like 2h to review the 1st one16:26
vincentfrancoisealexchadin: one last thing16:27
vincentfrancoisealexchadin: turn_host_to_acpi_s3_state doesn't exist yet16:27
vincentfrancoisealexchadin: when do you plan on pushing it? it's because you mention it in your worload_stabilization planner16:28
alexchadinvincentfrancoise: I think it requires some discussions16:28
alexchadinvincentfrancoise: sorry, but I have to go16:29
vincentfrancoisealexchadin: ok :) good evening16:30
alexchadinvincentfrancoise: have a nice evening!16:30
*** alexchadin has quit IRC16:31
chrisspencervincentfrancoise: I had a question about adding extra details to logs for bug: How I can/should go about verifying my fix?16:33
openstackLaunchpad bug 1642623 in watcher "Add resource ID in nova notification logs " [Wishlist,Triaged] - Assigned to Chris Spencer (christopher-m-spencer)16:33
vincentfrancoisechrisspencer: how can I help?16:34
vincentfrancoisechrisspencer: did you do a unit test to see if it get passed the message ID?16:34
chrisspencervincentfrancoise: I ran the existing unit tests but haven't written a new one16:35
vincentfrancoisechrisspencer: in TestReceiveNovaNotifications (watcher/tests/decision_engine/model/notification/
vincentfrancoisechrisspencer: I would add a new test that mocks the logger to see if the gets called with the right parameters and what not16:37
chrisspencervincentfrancoise: ack. will do16:38
vincentfrancoisechrisspencer: 1 test will probably do16:38
*** harlowja has joined #openstack-watcher16:39
acabotsballe_ : I'm back16:40
acabotsballe_ : 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
acabotsballe_ : fell free to suggest any new talk16: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 weeks16:47
acabotok will do16:47
sballe_acabot: I was going to submit the one on NN16:47
openstackgerritDavid TARDIVEL proposed openstack/watcher: New Applier Workflow Engine
sballe_hvprash: will be a co-speaker on taht one16:47
acabotsballe_ : great16:48
acabotsballe_ :
acabotI also opened the etherpad for PTG
sballe_tmetsch: should be there as well.16:50
*** harlowja has quit IRC17:33
*** vincentfrancoise has quit IRC17:34
*** tmetsch has quit IRC18:02
*** absubram has joined #openstack-watcher18:35
*** absubram has quit IRC19:17
*** adisky_ has quit IRC19:19
*** absubram has joined #openstack-watcher19:22
*** dtardivel has quit IRC20:24
*** wootehfoot has joined #openstack-watcher20:33
*** absubram has quit IRC20:43
*** absubram has joined #openstack-watcher21:00
*** absubram has quit IRC21:21
*** absubram has joined #openstack-watcher21:33
*** thorst has quit IRC22:07
*** thorst has joined #openstack-watcher22:39
*** thorst has quit IRC22:41
openstackgerritChris Spencer proposed openstack/watcher: Adding additional details to notification logs
*** absubram has quit IRC23:09
*** pshedimb has quit IRC23:24
*** acabot has quit IRC23:24
*** atuly_ has quit IRC23:24
*** pshedimb has joined #openstack-watcher23:24
*** acabot has joined #openstack-watcher23:24
*** atuly_ has joined #openstack-watcher23:24
*** wootehfoot has quit IRC23:28

Generated by 2.14.0 by Marius Gedminas - find it at!