*** jwcroppe has joined #openstack-powervm | 00:07 | |
*** newbie|2 has joined #openstack-powervm | 00:21 | |
*** newbie|2 has quit IRC | 00:25 | |
*** thorst_ has quit IRC | 00:30 | |
*** thorst_ has joined #openstack-powervm | 00:30 | |
*** thorst_ has quit IRC | 00:31 | |
*** jwcroppe has quit IRC | 00:32 | |
*** jwcroppe has joined #openstack-powervm | 00:33 | |
*** jwcroppe has quit IRC | 00:38 | |
*** kylek3h has joined #openstack-powervm | 00:48 | |
*** tlian has quit IRC | 01:15 | |
*** thorst_ has joined #openstack-powervm | 01:17 | |
*** thorst_ has quit IRC | 01:17 | |
openstackgerrit | Matt Rabe proposed openstack/nova-powervm master: Add vopt removal params to the power on job in spawn https://review.openstack.org/425780 | 01:22 |
---|---|---|
*** edmondsw has quit IRC | 01:22 | |
*** kylek3h has quit IRC | 01:23 | |
openstackgerrit | Matt Rabe proposed openstack/nova-powervm master: Add vopt removal params to the power on job in spawn https://review.openstack.org/425780 | 01:34 |
*** jwcroppe has joined #openstack-powervm | 01:50 | |
*** thorst_ has joined #openstack-powervm | 02:02 | |
*** esberglu has joined #openstack-powervm | 02:21 | |
*** esberglu has quit IRC | 02:26 | |
*** thorst_ has quit IRC | 02:29 | |
*** edmondsw has joined #openstack-powervm | 02:42 | |
*** edmondsw has quit IRC | 02:47 | |
*** mdrabe has quit IRC | 03:06 | |
*** esberglu has joined #openstack-powervm | 03:16 | |
*** tlian has joined #openstack-powervm | 03:19 | |
*** esberglu has quit IRC | 03:20 | |
*** thorst_ has joined #openstack-powervm | 03:29 | |
*** thorst_ has quit IRC | 03:30 | |
*** tlian has quit IRC | 04:23 | |
*** thorst_ has joined #openstack-powervm | 04:30 | |
*** thorst_ has quit IRC | 04:35 | |
*** newbie|2 has joined #openstack-powervm | 05:00 | |
*** esberglu has joined #openstack-powervm | 05:01 | |
*** esberglu has quit IRC | 05:05 | |
*** newbie|2 has quit IRC | 06:05 | |
*** tlian has joined #openstack-powervm | 06:11 | |
*** tlian has quit IRC | 06:15 | |
*** thorst_ has joined #openstack-powervm | 06:32 | |
*** thorst_ has quit IRC | 06:45 | |
*** esberglu has joined #openstack-powervm | 06:50 | |
*** esberglu has quit IRC | 06:55 | |
*** esberglu has joined #openstack-powervm | 08:40 | |
*** esberglu has quit IRC | 08:44 | |
*** esberglu has joined #openstack-powervm | 10:27 | |
*** esberglu has quit IRC | 10:32 | |
*** thorst_ has joined #openstack-powervm | 10:42 | |
*** thorst_ has quit IRC | 10:47 | |
*** smatzek has joined #openstack-powervm | 11:40 | |
*** k0da has joined #openstack-powervm | 11:45 | |
*** jpasqualetto has joined #openstack-powervm | 12:16 | |
*** svenkat has joined #openstack-powervm | 12:26 | |
*** thorst_ has joined #openstack-powervm | 12:45 | |
*** xia has joined #openstack-powervm | 13:12 | |
*** edmondsw has joined #openstack-powervm | 13:19 | |
*** edmondsw_ has joined #openstack-powervm | 13:21 | |
*** edmondsw has quit IRC | 13:23 | |
*** kylek3h has joined #openstack-powervm | 13:34 | |
*** kylek3h has quit IRC | 13:34 | |
*** kylek3h has joined #openstack-powervm | 13:35 | |
*** burgerk has joined #openstack-powervm | 13:48 | |
*** nbante has joined #openstack-powervm | 13:49 | |
*** jwcroppe has quit IRC | 13:52 | |
*** esberglu has joined #openstack-powervm | 14:00 | |
esberglu | #startmeeting powervm_driver_meeting | 14:00 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:00 |
thorst_ | yo - looks like efried will be out today | 14:00 |
thorst_ | he's apparently uber ill | 14:00 |
esberglu | Bummer | 14:01 |
thorst_ | yeah | 14:01 |
thorst_ | I can give a light status update from his side when we're ready in the meeting | 14:01 |
xia | o/ | 14:01 |
*** wangqwsh has joined #openstack-powervm | 14:02 | |
esberglu | #topic CI | 14:03 |
esberglu | I 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 patches | 14:04 |
esberglu | I wanted to test them concurrently on staging yesterday, couldn't with the network stuff | 14:04 |
esberglu | I've already tested the first one | 14:04 |
esberglu | Once those are both working, we are pretty close to turning on the IT publishing | 14:05 |
thorst_ | neat | 14:06 |
thorst_ | how bad were you hosed up with the lab issues? | 14:06 |
thorst_ | not sure if you saw my note | 14:06 |
*** AlexeyAbashkin has joined #openstack-powervm | 14:06 | |
*** jwcroppe has joined #openstack-powervm | 14:07 | |
esberglu | I just saw it now. Looks like I might be redeploying the environments today | 14:07 |
esberglu | I haven't checked the status of the systems yet today | 14:07 |
efried | o/ | 14:07 |
esberglu | You're alive! | 14:08 |
adreznec | Well if the controller got rebooted it's definitely unstacked | 14:08 |
adreznec | So there's that at least | 14:08 |
efried | barely | 14:08 |
*** mdrabe has joined #openstack-powervm | 14:08 | |
*** apearson has joined #openstack-powervm | 14:08 | |
esberglu | #action esberglu: Redeploy CI environments | 14:09 |
thorst_ | alright. Too bad that happened... | 14:09 |
thorst_ | :-( | 14:09 |
esberglu | #action esberglu: Test 2 CI patches | 14:09 |
efried | Looks like morpheus needs services started. | 14:09 |
esberglu | That's all I had for CI unless there are questions | 14:10 |
efried | oh | 14:10 |
efried | yeah | 14:10 |
efried | What's the scoop with ocata CI? | 14:10 |
thorst_ | efried: I sent a note to dom/julio about morpheus | 14:10 |
esberglu | I need to put a patch up. Not everything had a stable/ocata branch for a while | 14:10 |
*** jpasqualetto has quit IRC | 14:10 | |
efried | From 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 tempest | 14:10 |
thorst_ | it was also impacted...they need to make sure it can survive a reboot... | 14:10 |
efried | Okay, esberglu you got an action for that too? | 14:11 |
esberglu | #action: esberglu: Put up a patch for stable/ocata tempest in CI | 14:11 |
esberglu | Yeah, super easy. We can get it in before I redeploy prod. | 14:12 |
efried | sweet | 14:12 |
efried | #topic In-tree change sets? | 14:13 |
esberglu | Yep | 14:14 |
efried | Right 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 |
efried | Once that's done, I'll flush those changes back into in-tree change #2 (start/stop). | 14:15 |
efried | And 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 |
efried | Yeah, 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 good | 14:16 |
thorst_ | assuming you're able | 14:16 |
thorst_ | #action adreznec thorst Review new patch set pushed up by efried | 14:17 |
efried | Okay, https://review.openstack.org/#/c/391288/ is rebased, ready for review. | 14:18 |
wangqwsh | yes, just updated the comments from reviewers | 14:19 |
efried | wangqwsh - you put up a couple new patch sets on localdisk - what were those about? (Haven't looked yet) | 14:20 |
wangqwsh | let me check it | 14:20 |
efried | oh, looks like it may have just been a rebase on the first one. | 14:21 |
wangqwsh | updated the uuid with uuidutils.generate_uuid method to create | 14:21 |
wangqwsh | this is from Ed Leafe review in spawn/destory change set | 14:22 |
*** jpasqualetto has joined #openstack-powervm | 14:23 | |
efried | The 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 |
efried | done here. | 14:24 |
thorst_ | #action thorst to review mdrabe's change set | 14:24 |
efried | #link https://review.openstack.org/#/c/425780/ | 14:25 |
efried | That's all I have. | 14:25 |
esberglu | Any one else have topics? | 14:27 |
thorst_ | I'm good | 14:27 |
wangqwsh | sorry, the neo4 can not connect | 14:27 |
*** smatzek has quit IRC | 14:27 | |
efried | Looks like morpheus is back. | 14:27 |
thorst_ | wangqwsh: can you get to the FSP? | 14:27 |
adreznec | Might have shut off with the power outage | 14:27 |
wangqwsh | no... | 14:27 |
wangqwsh | which fsp? | 14:28 |
thorst_ | let me look | 14:28 |
wangqwsh | thanks :) | 14:28 |
thorst_ | ooo, yikes. Yeah, it didn't boot properly | 14:29 |
thorst_ | fixing... | 14:29 |
*** apearson has quit IRC | 14:30 | |
esberglu | Reminder: This week I will be tagging the final Release Candidates (friday). The Ocata release is Feb. 22 | 14:31 |
*** dwayne has quit IRC | 14:32 | |
esberglu | Sounds like no one else has any topics. Thanks for joining | 14:32 |
*** apearson has joined #openstack-powervm | 14:32 | |
esberglu | #endmeeting | 14:32 |
openstack | Meeting ended Tue Feb 14 14:32:34 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:32 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-02-14-14.00.html | 14:32 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-02-14-14.00.txt | 14:32 |
openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-02-14-14.00.log.html | 14:32 |
*** xia has quit IRC | 14:38 | |
thorst_ | wangqwsh: I think its running now | 14:46 |
thorst_ | please verify | 14:46 |
wangqwsh | yes, works! thanks! | 14:46 |
*** smatzek has joined #openstack-powervm | 14:51 | |
*** tlian has joined #openstack-powervm | 14:52 | |
*** apearson has quit IRC | 14:56 | |
*** burgerk has quit IRC | 14:57 | |
*** apearson has joined #openstack-powervm | 14:58 | |
*** AlexeyAbashkin has quit IRC | 15:08 | |
*** tblakes has joined #openstack-powervm | 15:11 | |
*** kriskend has joined #openstack-powervm | 15:17 | |
*** tjakobs has joined #openstack-powervm | 15:25 | |
*** burgerk has joined #openstack-powervm | 15:25 | |
*** apearson has quit IRC | 15:40 | |
*** apearson has joined #openstack-powervm | 15:42 | |
efried | esberglu Presumably https://review.openstack.org/#/c/425780/ is failing because the CI env is still being rebuilt? (mdrabe) | 15:53 |
esberglu | Yep | 15:53 |
*** dwayne has joined #openstack-powervm | 16:04 | |
*** tblakes has quit IRC | 16:16 | |
*** mdrabe has quit IRC | 16:25 | |
*** mdrabe has joined #openstack-powervm | 16:42 | |
mdrabe | esberglu 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 |
esberglu | CI 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 |
esberglu | I will put a recheck on there when it is up | 16:47 |
mdrabe | Ah ok thanks | 16:48 |
*** tblakes has joined #openstack-powervm | 17:00 | |
*** jpasqualetto has quit IRC | 17:24 | |
*** jpasqualetto has joined #openstack-powervm | 17:44 | |
*** k0da has quit IRC | 18:23 | |
*** nbante has quit IRC | 18:57 | |
*** seroyer has joined #openstack-powervm | 19:35 | |
*** k0da has joined #openstack-powervm | 20:00 | |
*** openstackgerrit has quit IRC | 20:18 | |
mdrabe | esberglu: Looks like the CI failed one test: neutron.tests.tempest.scenario.test_trunk | 20:38 |
mdrabe | I'm fairly sure that's unrelated to my change, is it perhaps that neutron isn't on the right branch? | 20:38 |
esberglu | mdrabe: Yep. Thats a new test that isn't supported. I just added it to the skip list | 20:42 |
esberglu | Another recheck should do the trick | 20:42 |
mdrabe | k I just dropped it, thanks | 20:42 |
*** jpasqualetto has quit IRC | 20:46 | |
*** jpasqualetto has joined #openstack-powervm | 20:47 | |
*** openstackgerrit has joined #openstack-powervm | 21:00 | |
openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Refactor and synchronize power ops https://review.openstack.org/433927 | 21:00 |
efried | thorst_ ^^ | 21:01 |
thorst_ | I feel like that's two change sets in one...right off the bat. | 21:02 |
thorst_ | 1) add locking | 21:02 |
thorst_ | 2) refactor | 21:02 |
efried | thorst_ I considered that, but take a look - those two things are pretty well intertwined. | 21:03 |
efried | I could split it out if I had to. | 21:03 |
efried | But I would hate you for it. | 21:03 |
efried | No cookie. | 21:03 |
*** tblakes has quit IRC | 21:04 | |
thorst_ | hates a strong word hombre | 21:05 |
thorst_ | I was thinking the lock thing could kick in immediately...while we argue through the power refactor | 21:06 |
thorst_ | but let me look at that | 21:06 |
thorst_ | also, aren't you currently dying? | 21:06 |
efried | thorst_ 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 |
efried | But 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 |
efried | Won't help streamline the process any, though. | 21:08 |
thorst_ | let me review first | 21:09 |
efried | ack | 21:09 |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 21:09 |
thorst_ | you're making me feel bad if I say break it out when you are so sick | 21:09 |
thorst_ | :-) | 21:09 |
efried | thorst_ It's no biggie. I was thinking about how hard it would be if I had to do the locking first. | 21:14 |
efried | I think it's probably the right thing to do. | 21:14 |
thorst_ | see my note | 21:14 |
thorst_ | I think that fix makes sense for Ocata too | 21:14 |
thorst_ | so a bug report. | 21:14 |
efried | thorst_ The synchronization fix, not the refactor, right? | 21:15 |
thorst_ | at least that back to Ocata | 21:15 |
efried | Ugh, forget I said that. | 21:15 |
efried | What pypowervm are we going to have for ocata? thorst_ esberglu adreznec | 21:15 |
efried | .5, which will include the power refactor? | 21:16 |
thorst_ | yep | 21:18 |
thorst_ | there are already deps for that in OOT | 21:18 |
thorst_ | but not IT | 21:18 |
efried | ugh, so I'm going to have to wait a bit to do some of this IT. | 21:19 |
thorst_ | correct. | 21:19 |
efried | Actually, a very small percentage of it will have to wait. No biggie. | 21:19 |
efried | thorst_ https://bugs.launchpad.net/nova-powervm/+bug/1664718 https://bugs.launchpad.net/nova-powervm/+bug/1664720 | 21:22 |
openstack | Launchpad bug 1664718 in nova-powervm "Refactor deprecated power_on add_parms" [Low,In progress] - Assigned to Eric Fried (efried) | 21:22 |
openstack | Launchpad bug 1664720 in nova-powervm "Synchronize power ops" [Medium,In progress] - Assigned to Eric Fried (efried) | 21:22 |
thorst_ | efried: rockin | 21:22 |
thorst_ | but maybe for tomorrow | 21:22 |
thorst_ | after you're not dying? | 21:22 |
efried | thorst_ Want me to split 'em? | 21:22 |
thorst_ | well, not for OOT | 21:23 |
thorst_ | but IT, sure | 21:23 |
thorst_ | not sure you even need to call it out IT | 21:23 |
efried | I'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 phone | 21:26 |
openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Refactor power ops https://review.openstack.org/433927 | 21:29 |
*** smatzek has quit IRC | 21:30 | |
openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Synchronize power ops https://review.openstack.org/433935 | 21:32 |
efried | thorst_ ^^ Split is done. Will wait to cherry-pick until reviewed. | 21:32 |
efried | oh, and you're right about refetching the entry. Lemme fix that now. | 21:35 |
efried | I'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 there | 21:36 |
thorst_ | since, even with locking, it could be a problem if you're looking at an old wrapper | 21:37 |
efried | yup. | 21:37 |
efried | Can'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 completely | 21:39 |
thorst_ | just always do a xag-less query | 21: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 IRC | 21:40 | |
*** kriskend has quit IRC | 21:55 | |
*** kriskend has joined #openstack-powervm | 21:55 | |
*** svenkat has quit IRC | 21:57 | |
esberglu | mdrabe: Your change made it through CI | 21:59 |
mdrabe | Yep I saw, thorst_ https://review.openstack.org/#/c/425780/7 pls? | 22:00 |
*** kriskend has quit IRC | 22:00 | |
*** burgerk_ has joined #openstack-powervm | 22:05 | |
*** burgerk has quit IRC | 22:06 | |
*** tblakes has joined #openstack-powervm | 22:09 | |
*** burgerk_ has quit IRC | 22:12 | |
*** thorst_ has quit IRC | 22:15 | |
*** thorst_ has joined #openstack-powervm | 22:24 | |
*** thorst_ has quit IRC | 22:28 | |
*** edmondsw_ has quit IRC | 22:33 | |
*** seroyer has quit IRC | 22:34 | |
*** smatzek has joined #openstack-powervm | 22:35 | |
*** tblakes_ has joined #openstack-powervm | 22:35 | |
*** edmondsw has joined #openstack-powervm | 22:35 | |
*** tonyb_ has joined #openstack-powervm | 22:38 | |
*** edmondsw has quit IRC | 22:40 | |
*** dwayne has quit IRC | 22:40 | |
*** tblakes has quit IRC | 22:42 | |
*** efried has quit IRC | 22:42 | |
*** tonyb has quit IRC | 22:42 | |
openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Synchronize power ops https://review.openstack.org/433935 | 22:45 |
*** tonyb_ is now known as tonyb | 22:46 | |
*** efried has joined #openstack-powervm | 22:51 | |
*** smatzek has quit IRC | 22:53 | |
*** esberglu has quit IRC | 22:54 | |
*** esberglu has joined #openstack-powervm | 22:54 | |
*** esberglu has quit IRC | 22:58 | |
*** mdrabe has quit IRC | 23:06 | |
*** esberglu has joined #openstack-powervm | 23:07 | |
*** seroyer has joined #openstack-powervm | 23:11 | |
*** esberglu has quit IRC | 23:11 | |
*** seroyer has quit IRC | 23:13 | |
*** thorst_ has joined #openstack-powervm | 23:19 | |
*** apearson has quit IRC | 23:20 | |
*** thorst_ has quit IRC | 23:20 | |
*** tjakobs has quit IRC | 23:31 | |
*** jwcroppe has quit IRC | 23:52 | |
*** k0da has quit IRC | 23:53 | |
*** dwayne has joined #openstack-powervm | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!