Tuesday, 2017-02-14

*** jwcroppe has joined #openstack-powervm00:07
*** newbie|2 has joined #openstack-powervm00:21
*** newbie|2 has quit IRC00:25
*** thorst_ has quit IRC00:30
*** thorst_ has joined #openstack-powervm00:30
*** thorst_ has quit IRC00:31
*** jwcroppe has quit IRC00:32
*** jwcroppe has joined #openstack-powervm00:33
*** jwcroppe has quit IRC00:38
*** kylek3h has joined #openstack-powervm00:48
*** tlian has quit IRC01:15
*** thorst_ has joined #openstack-powervm01:17
*** thorst_ has quit IRC01:17
openstackgerritMatt Rabe proposed openstack/nova-powervm master: Add vopt removal params to the power on job in spawn  https://review.openstack.org/42578001:22
*** edmondsw has quit IRC01:22
*** kylek3h has quit IRC01:23
openstackgerritMatt Rabe proposed openstack/nova-powervm master: Add vopt removal params to the power on job in spawn  https://review.openstack.org/42578001:34
*** jwcroppe has joined #openstack-powervm01:50
*** thorst_ has joined #openstack-powervm02:02
*** esberglu has joined #openstack-powervm02:21
*** esberglu has quit IRC02:26
*** thorst_ has quit IRC02:29
*** edmondsw has joined #openstack-powervm02:42
*** edmondsw has quit IRC02:47
*** mdrabe has quit IRC03:06
*** esberglu has joined #openstack-powervm03:16
*** tlian has joined #openstack-powervm03:19
*** esberglu has quit IRC03:20
*** thorst_ has joined #openstack-powervm03:29
*** thorst_ has quit IRC03:30
*** tlian has quit IRC04:23
*** thorst_ has joined #openstack-powervm04:30
*** thorst_ has quit IRC04:35
*** newbie|2 has joined #openstack-powervm05:00
*** esberglu has joined #openstack-powervm05:01
*** esberglu has quit IRC05:05
*** newbie|2 has quit IRC06:05
*** tlian has joined #openstack-powervm06:11
*** tlian has quit IRC06:15
*** thorst_ has joined #openstack-powervm06:32
*** thorst_ has quit IRC06:45
*** esberglu has joined #openstack-powervm06:50
*** esberglu has quit IRC06:55
*** esberglu has joined #openstack-powervm08:40
*** esberglu has quit IRC08:44
*** esberglu has joined #openstack-powervm10:27
*** esberglu has quit IRC10:32
*** thorst_ has joined #openstack-powervm10:42
*** thorst_ has quit IRC10:47
*** smatzek has joined #openstack-powervm11:40
*** k0da has joined #openstack-powervm11:45
*** jpasqualetto has joined #openstack-powervm12:16
*** svenkat has joined #openstack-powervm12:26
*** thorst_ has joined #openstack-powervm12:45
*** xia has joined #openstack-powervm13:12
*** edmondsw has joined #openstack-powervm13:19
*** edmondsw_ has joined #openstack-powervm13:21
*** edmondsw has quit IRC13:23
*** kylek3h has joined #openstack-powervm13:34
*** kylek3h has quit IRC13:34
*** kylek3h has joined #openstack-powervm13:35
*** burgerk has joined #openstack-powervm13:48
*** nbante has joined #openstack-powervm13:49
*** jwcroppe has quit IRC13:52
*** esberglu has joined #openstack-powervm14:00
esberglu#startmeeting powervm_driver_meeting14:00
openstackMeeting started Tue Feb 14 14:00:34 2017 UTC and is due to finish in 60 minutes.  The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
openstackThe meeting name has been set to 'powervm_driver_meeting'14:00
thorst_yo - looks like efried will be out today14:00
thorst_he's apparently uber ill14:00
esbergluBummer14:01
thorst_yeah14:01
thorst_I can give a light status update from his side when we're ready in the meeting14:01
xiao/14:01
*** wangqwsh has joined #openstack-powervm14:02
esberglu#topic CI14:03
esbergluI have two patches out for neo-os-ci. The first is so OOT doesn't run on IT patches. The second is so all IT runs pick up the IT patches14:04
esbergluI wanted to test them concurrently on staging yesterday, couldn't with the network stuff14:04
esbergluI've already tested the first one14:04
esbergluOnce those are both working, we are pretty close to turning on the IT publishing14:05
thorst_neat14:06
thorst_how bad were you hosed up with the lab issues?14:06
thorst_not sure if you saw my note14:06
*** AlexeyAbashkin has joined #openstack-powervm14:06
*** jwcroppe has joined #openstack-powervm14:07
esbergluI just saw it now. Looks like I might be redeploying the environments today14:07
esbergluI haven't checked the status of the systems yet today14:07
efriedo/14:07
esbergluYou're alive!14:08
adreznecWell if the controller got rebooted it's definitely unstacked14:08
adreznecSo there's that at least14:08
efriedbarely14:08
*** mdrabe has joined #openstack-powervm14:08
*** apearson has joined #openstack-powervm14:08
esberglu#action esberglu: Redeploy CI environments14:09
thorst_alright.  Too bad that happened...14:09
thorst_:-(14:09
esberglu#action esberglu: Test 2 CI patches14:09
efriedLooks like morpheus needs services started.14:09
esbergluThat's all I had for CI unless there are questions14:10
efriedoh14:10
efriedyeah14:10
efriedWhat's the scoop with ocata CI?14:10
thorst_efried: I sent a note to dom/julio about morpheus14:10
esbergluI need to put a patch up. Not everything had a stable/ocata branch for a while14:10
*** jpasqualetto has quit IRC14:10
efriedFrom last night:  <efried> esberglu When you get a chance (like tomorrow) can you look at https://review.openstack.org/#/c/431583/ ?14:10
efried<efried> This is failing OOT consistently.  It might be the first change set we've seen explicitly in the stable/ocata branch.14:10
efried<efried> It may be that we need to do something to get that branch working.14:10
efried<efried> Yeah - 12:35:23 stable/ocata is not a supported branch for tempest14:10
thorst_it was also impacted...they need to make sure it can survive a reboot...14:10
efriedOkay, esberglu you got an action for that too?14:11
esberglu#action: esberglu: Put up a patch for stable/ocata tempest in CI14:11
esbergluYeah, super easy. We can get it in before I redeploy prod.14:12
efriedsweet14:12
efried#topic In-tree change sets?14:13
esbergluYep14:14
efriedRight now I'm working on refactoring the usage of power on/off in the OOT driver, including consolidating so everything runs through vm.py and synchronizing all those methods on the instance ID.  This may resolve some of the timing failures we were seeing in the OOT CI.14:14
efriedOnce that's done, I'll flush those changes back into in-tree change #2 (start/stop).14:15
efriedAnd do the rebase string.14:15
thorst_adreznec and I to start reviewing?  Or a day or so yet (two days?  because of illness)14:15
efriedYeah, well, I can rebase #1 real quickly and you guys can review that one.  Then yeah, it'll take me a day or two to get #2 piled on top.14:16
thorst_yep...that'd be good14:16
thorst_assuming you're able14:16
thorst_#action adreznec thorst Review new patch set pushed up by efried14:17
efriedOkay, https://review.openstack.org/#/c/391288/ is rebased, ready for review.14:18
wangqwshyes, just updated the comments from reviewers14:19
efriedwangqwsh - you put up a couple new patch sets on localdisk - what were those about?  (Haven't looked yet)14:20
wangqwshlet me check it14:20
efriedoh, looks like it may have just been a rebase on the first one.14:21
wangqwshupdated the uuid with uuidutils.generate_uuid method to create14:21
wangqwshthis is from Ed Leafe review in spawn/destory change set14:22
*** jpasqualetto has joined #openstack-powervm14:23
efriedThe only other thing I have on driver code is mdrabe's change set, which looks like it needs another review from me & thorst_14:23
efrieddone here.14:24
thorst_#action thorst to review mdrabe's change set14:24
efried#link https://review.openstack.org/#/c/425780/14:25
efriedThat's all I have.14:25
esbergluAny one else have topics?14:27
thorst_I'm good14:27
wangqwshsorry, the neo4 can not connect14:27
*** smatzek has quit IRC14:27
efriedLooks like morpheus is back.14:27
thorst_wangqwsh: can you get to the FSP?14:27
adreznecMight have shut off with the power outage14:27
wangqwshno...14:27
wangqwshwhich fsp?14:28
thorst_let me look14:28
wangqwshthanks :)14:28
thorst_ooo, yikes.  Yeah, it didn't boot properly14:29
thorst_fixing...14:29
*** apearson has quit IRC14:30
esbergluReminder: This week I will be tagging the final Release Candidates (friday). The Ocata release is Feb. 2214:31
*** dwayne has quit IRC14:32
esbergluSounds like no one else has any topics. Thanks for joining14:32
*** apearson has joined #openstack-powervm14:32
esberglu#endmeeting14:32
openstackMeeting ended Tue Feb 14 14:32:34 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:32
openstackMinutes:        http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-02-14-14.00.html14:32
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-02-14-14.00.txt14:32
openstackLog:            http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-02-14-14.00.log.html14:32
*** xia has quit IRC14:38
thorst_wangqwsh: I think its running now14:46
thorst_please verify14:46
wangqwshyes, works! thanks!14:46
*** smatzek has joined #openstack-powervm14:51
*** tlian has joined #openstack-powervm14:52
*** apearson has quit IRC14:56
*** burgerk has quit IRC14:57
*** apearson has joined #openstack-powervm14:58
*** AlexeyAbashkin has quit IRC15:08
*** tblakes has joined #openstack-powervm15:11
*** kriskend has joined #openstack-powervm15:17
*** tjakobs has joined #openstack-powervm15:25
*** burgerk has joined #openstack-powervm15:25
*** apearson has quit IRC15:40
*** apearson has joined #openstack-powervm15:42
efriedesberglu Presumably https://review.openstack.org/#/c/425780/ is failing because the CI env is still being rebuilt?  (mdrabe)15:53
esbergluYep15:53
*** dwayne has joined #openstack-powervm16:04
*** tblakes has quit IRC16:16
*** mdrabe has quit IRC16:25
*** mdrabe has joined #openstack-powervm16:42
mdrabeesberglu Does the CI take some time to publish results? Saw it failed for https://review.openstack.org/#/c/425780/7 but when I click on it I'm getting "File Not Found"16:45
esbergluCI still isn't back up from the outage yesterday. When they rebooted the CI mgmt server, some of the services started back up, but not all. So it was behaving really weird.16:47
esbergluI will put a recheck on there when it is up16:47
mdrabeAh ok thanks16:48
*** tblakes has joined #openstack-powervm17:00
*** jpasqualetto has quit IRC17:24
*** jpasqualetto has joined #openstack-powervm17:44
*** k0da has quit IRC18:23
*** nbante has quit IRC18:57
*** seroyer has joined #openstack-powervm19:35
*** k0da has joined #openstack-powervm20:00
*** openstackgerrit has quit IRC20:18
mdrabeesberglu: Looks like the CI failed one test: neutron.tests.tempest.scenario.test_trunk20:38
mdrabeI'm fairly sure that's unrelated to my change, is it perhaps that neutron isn't on the right branch?20:38
esberglumdrabe: Yep. Thats a new test that isn't supported. I just added it to the skip list20:42
esbergluAnother recheck should do the trick20:42
mdrabek I just dropped it, thanks20:42
*** jpasqualetto has quit IRC20:46
*** jpasqualetto has joined #openstack-powervm20:47
*** openstackgerrit has joined #openstack-powervm21:00
openstackgerritEric Fried proposed openstack/nova-powervm master: Refactor and synchronize power ops  https://review.openstack.org/43392721:00
efriedthorst_ ^^21:01
thorst_I feel like that's two change sets in one...right off the bat.21:02
thorst_1) add locking21:02
thorst_2) refactor21:02
efriedthorst_ I considered that, but take a look - those two things are pretty well intertwined.21:03
efriedI could split it out if I had to.21:03
efriedBut I would hate you for it.21:03
efriedNo cookie.21:03
*** tblakes has quit IRC21:04
thorst_hates a strong word hombre21:05
thorst_I was thinking the lock thing could kick in immediately...while we argue through the power refactor21:06
thorst_but let me look at that21:06
thorst_also, aren't you currently dying?21:06
efriedthorst_ A loving kind of hate.  Actually, to do the locking first would have been *much* harder.  Because some places we were calling power_on/_off directly to pypowervm, some places through vm.py, some places through the Tasks (and the Tasks themselves were doing some of each).  So in order to get the locking down to just three locations (power_on, power_off, and reboot), I had to do the refactor first.21:07
efriedBut now that I've done all of this, I could rip the locking out of this change set and stuff it in a subsequent one, if that's important to you.21:08
efriedWon't help streamline the process any, though.21:08
thorst_let me review first21:09
efriedack21:09
openstackgerritEric Berglund proposed openstack/nova-powervm master: DNM: CI Check2  https://review.openstack.org/32831721:09
thorst_you're making me feel bad if I say break it out when you are so sick21:09
thorst_:-)21:09
efriedthorst_ It's no biggie.  I was thinking about how hard it would be if I had to do the locking first.21:14
efriedI think it's probably the right thing to do.21:14
thorst_see my note21:14
thorst_I think that fix makes sense for Ocata too21:14
thorst_so a bug report.21:14
efriedthorst_ The synchronization fix, not the refactor, right?21:15
thorst_at least that back to Ocata21:15
efriedUgh, forget I said that.21:15
efriedWhat pypowervm are we going to have for ocata?  thorst_ esberglu adreznec21:15
efried.5, which will include the power refactor?21:16
thorst_yep21:18
thorst_there are already deps for that in OOT21:18
thorst_but not IT21:18
efriedugh, so I'm going to have to wait a bit to do some of this IT.21:19
thorst_correct.21:19
efriedActually, a very small percentage of it will have to wait.  No biggie.21:19
efriedthorst_ https://bugs.launchpad.net/nova-powervm/+bug/1664718 https://bugs.launchpad.net/nova-powervm/+bug/166472021:22
openstackLaunchpad bug 1664718 in nova-powervm "Refactor deprecated power_on add_parms" [Low,In progress] - Assigned to Eric Fried (efried)21:22
openstackLaunchpad bug 1664720 in nova-powervm "Synchronize power ops" [Medium,In progress] - Assigned to Eric Fried (efried)21:22
thorst_efried: rockin21:22
thorst_but maybe for tomorrow21:22
thorst_after you're not dying?21:22
efriedthorst_ Want me to split 'em?21:22
thorst_well, not for OOT21:23
thorst_but IT, sure21:23
thorst_not sure you even need to call it out IT21:23
efriedI'll still be dying tomorrow.  Colds like this last a solid week for me.  I just haven't had one in a while.21:23
thorst_yeah, you sounded horrible on the phone21:26
openstackgerritEric Fried proposed openstack/nova-powervm master: Refactor power ops  https://review.openstack.org/43392721:29
*** smatzek has quit IRC21:30
openstackgerritEric Fried proposed openstack/nova-powervm master: Synchronize power ops  https://review.openstack.org/43393521:32
efriedthorst_ ^^ Split is done.  Will wait to cherry-pick until reviewed.21:32
efriedoh, and you're right about refetching the entry.  Lemme fix that now.21:35
efriedI'll do that in the locking change set.  Otherwise doesn't make a lot of difference.21:36
thorst_right.21:36
thorst_I thought it made more sense there21:36
thorst_since, even with locking, it could be a problem if you're looking at an old wrapper21:37
efriedyup.21:37
efriedCan't just look up the qp, tho.  The pypowervm op needs the wrapper.21:39
thorst_I wonder if we should just pull that out completely21:39
thorst_just always do a xag-less query21:39
thorst_get the latest.21:39
thorst_since we are having issues...I'd rather take the 50 ms hit and just get it.21:40
*** jpasqualetto has quit IRC21:40
*** kriskend has quit IRC21:55
*** kriskend has joined #openstack-powervm21:55
*** svenkat has quit IRC21:57
esberglumdrabe: Your change made it through CI21:59
mdrabeYep I saw, thorst_ https://review.openstack.org/#/c/425780/7 pls?22:00
*** kriskend has quit IRC22:00
*** burgerk_ has joined #openstack-powervm22:05
*** burgerk has quit IRC22:06
*** tblakes has joined #openstack-powervm22:09
*** burgerk_ has quit IRC22:12
*** thorst_ has quit IRC22:15
*** thorst_ has joined #openstack-powervm22:24
*** thorst_ has quit IRC22:28
*** edmondsw_ has quit IRC22:33
*** seroyer has quit IRC22:34
*** smatzek has joined #openstack-powervm22:35
*** tblakes_ has joined #openstack-powervm22:35
*** edmondsw has joined #openstack-powervm22:35
*** tonyb_ has joined #openstack-powervm22:38
*** edmondsw has quit IRC22:40
*** dwayne has quit IRC22:40
*** tblakes has quit IRC22:42
*** efried has quit IRC22:42
*** tonyb has quit IRC22:42
openstackgerritEric Fried proposed openstack/nova-powervm master: Synchronize power ops  https://review.openstack.org/43393522:45
*** tonyb_ is now known as tonyb22:46
*** efried has joined #openstack-powervm22:51
*** smatzek has quit IRC22:53
*** esberglu has quit IRC22:54
*** esberglu has joined #openstack-powervm22:54
*** esberglu has quit IRC22:58
*** mdrabe has quit IRC23:06
*** esberglu has joined #openstack-powervm23:07
*** seroyer has joined #openstack-powervm23:11
*** esberglu has quit IRC23:11
*** seroyer has quit IRC23:13
*** thorst_ has joined #openstack-powervm23:19
*** apearson has quit IRC23:20
*** thorst_ has quit IRC23:20
*** tjakobs has quit IRC23:31
*** jwcroppe has quit IRC23:52
*** k0da has quit IRC23:53
*** dwayne has joined #openstack-powervm23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!