| *** svenkat has joined #openstack-powervm | 00:58 | |
| *** kriskend has quit IRC | 01:16 | |
| *** apearson has joined #openstack-powervm | 01:43 | |
| *** thorst has quit IRC | 02:43 | |
| *** thorst has joined #openstack-powervm | 02:44 | |
| *** svenkat has quit IRC | 02:45 | |
| -openstackstatus- NOTICE: Jobs running on osic nodes are failing due to network issues with the mirror. We are temporarily disabling the cloud. | 02:48 | |
| *** thorst has quit IRC | 02:52 | |
| *** thorst has joined #openstack-powervm | 03:50 | |
| *** seroyer has quit IRC | 03:52 | |
| *** thorst has quit IRC | 03:58 | |
| *** thorst has joined #openstack-powervm | 04:55 | |
| *** thorst has quit IRC | 05:03 | |
| *** k0da has quit IRC | 05:44 | |
| *** thorst has joined #openstack-powervm | 06:01 | |
| *** thorst has quit IRC | 06:07 | |
| *** thorst has joined #openstack-powervm | 07:05 | |
| *** thorst has quit IRC | 07:13 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:14 | |
| *** Alexey_Abashkin_ has quit IRC | 07:17 | |
| *** k0da has joined #openstack-powervm | 07:40 | |
| *** thorst has joined #openstack-powervm | 08:12 | |
| *** thorst has quit IRC | 08:17 | |
| *** burgerk_ has joined #openstack-powervm | 08:32 | |
| *** dwayne has quit IRC | 08:41 | |
| *** burgerk has quit IRC | 08:41 | |
| *** toan has quit IRC | 08:41 | |
| *** dwayne has joined #openstack-powervm | 08:42 | |
| *** toan has joined #openstack-powervm | 08:49 | |
| *** thorst has joined #openstack-powervm | 09:15 | |
| *** thorst has quit IRC | 09:23 | |
| *** openstack has joined #openstack-powervm | 09:30 | |
| *** thorst has joined #openstack-powervm | 10:20 | |
| *** thorst has quit IRC | 10:28 | |
| *** seroyer has joined #openstack-powervm | 11:25 | |
| *** thorst has joined #openstack-powervm | 11:26 | |
| *** thorst has quit IRC | 11:32 | |
| *** openstackgerrit has quit IRC | 11:48 | |
| *** openstackgerrit has joined #openstack-powervm | 11:49 | |
| *** thorst has joined #openstack-powervm | 11:54 | |
| *** k0da has quit IRC | 12:02 | |
| *** seroyer has quit IRC | 12:13 | |
| *** edmondsw has joined #openstack-powervm | 12:14 | |
| *** openstackgerrit has quit IRC | 12:18 | |
| *** openstackgerrit has joined #openstack-powervm | 12:18 | |
| *** apearson has quit IRC | 12:21 | |
| *** k0da has joined #openstack-powervm | 12:29 | |
| *** svenkat has joined #openstack-powervm | 12:41 | |
| *** seroyer has joined #openstack-powervm | 12:58 | |
| *** mdrabe has joined #openstack-powervm | 12:58 | |
| *** apearson has joined #openstack-powervm | 12:59 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: WIP: Pre-allocate trunk adapter on target host https://review.openstack.org/350231 | 12:59 |
|---|---|---|
| *** efried has joined #openstack-powervm | 13:00 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm: DNM: ci check https://review.openstack.org/328315 | 13:05 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm: DNM: CI Check2 https://review.openstack.org/328317 | 13:05 |
| *** tblakes has joined #openstack-powervm | 13:06 | |
| *** tblakes_ has joined #openstack-powervm | 13:10 | |
| *** tblakes has quit IRC | 13:10 | |
| *** tblakes_ is now known as tblakes | 13:10 | |
| *** esberglu has joined #openstack-powervm | 13:39 | |
| *** k0da has quit IRC | 13:45 | |
| *** k0da has joined #openstack-powervm | 13:53 | |
| *** tjakobs has joined #openstack-powervm | 14:50 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Pre-allocate trunk adapter on target host https://review.openstack.org/350231 | 15:12 |
| thorst | efried: ^^ Can you review that today? | 15:12 |
| thorst | there's still one piece of code to do... | 15:12 |
| thorst | o, and I need to fix pep8...let me do that quick | 15:12 |
| efried | thorst, ack | 15:13 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Pre-allocate trunk adapter on target host https://review.openstack.org/350231 | 15:15 |
| thorst | efried: ^^ that one passes tox at least | 15:15 |
| efried | thorst, roger wilco. | 15:15 |
| seroyer | thorst, with that change, that means that the trunk adapter is pre-created with the right PVID before the migration starts? | 15:20 |
| thorst | seroyer: yes | 15:23 |
| thorst | I'm just adding some rollback logic now, in case the migration fails. | 15:24 |
| thorst | it is not the change you and I were discussing about detecting the resume. | 15:24 |
| seroyer | Cool. | 15:24 |
| thorst | that's a whole bigger thing... | 15:24 |
| thorst | this roll back code is one of the more pleasant aspects of this live migrate bit. | 15:29 |
| *** k0da has quit IRC | 15:44 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Pre-allocate trunk adapter on target host https://review.openstack.org/350231 | 16:21 |
| thorst | efried seroyer kriskend adreznec: ^^ Now with rollback. Would really appreciate reviews today so that we can put this to rest... | 16:21 |
| efried | ack | 16:22 |
| *** dwayne has quit IRC | 16:41 | |
| *** apearson has quit IRC | 16:49 | |
| *** dwayne has joined #openstack-powervm | 17:04 | |
| *** apearson has joined #openstack-powervm | 17:08 | |
| thorst | svenkat: can you see latest comment https://review.openstack.org/#/c/331707/ | 17:34 |
| thorst | and weigh in on the new proposal...seems like its getting good traction... | 17:34 |
| svenkat | @thorst, i got an email on it, i noticed it. there is alternate solution with events. i will provide my comments | 17:37 |
| *** esberglu_ has joined #openstack-powervm | 17:41 | |
| *** esberglu has quit IRC | 17:41 | |
| *** esberglu_ has quit IRC | 17:41 | |
| *** esberglu has joined #openstack-powervm | 17:44 | |
| *** esberglu_ has joined #openstack-powervm | 17:46 | |
| *** esberglu_ has left #openstack-powervm | 17:50 | |
| *** esberglu_ has joined #openstack-powervm | 17:51 | |
| *** esberglu_ has quit IRC | 17:52 | |
| *** thorst has quit IRC | 18:48 | |
| *** k0da has joined #openstack-powervm | 18:49 | |
| *** thorst has joined #openstack-powervm | 19:13 | |
| *** miltonm has quit IRC | 19:14 | |
| *** mdrabe has quit IRC | 19:17 | |
| *** mdrabe has joined #openstack-powervm | 19:18 | |
| *** miltonm has joined #openstack-powervm | 19:19 | |
| thorst | efried: we're blocked on powervm changes...due to a new pep8 rule | 19:34 |
| thorst | I'm getting a fix in | 19:34 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Fix code for new pep8 rules https://review.openstack.org/385132 | 19:38 |
| thorst | ^^ | 19:38 |
| efried | thorst, is this pep8 or hacking? | 19:47 |
| efried | thorst, ya know, we might could put our config options into our nova/conf namespace. | 19:49 |
| efried | thorst, see review | 20:07 |
| thorst | hmmm... | 20:18 |
| thorst | that seems like a second change set? | 20:18 |
| efried | thorst, how so? We're fixing the failure rather than ignoring it. | 20:26 |
| efried | Or did you just want to have some time to burn it in? | 20:26 |
| thorst | time to burn that in | 20:29 |
| thorst | I don't like putting things in the nova namespace personally | 20:29 |
| thorst | unless we have to | 20:29 |
| thorst | it creates overlap issues | 20:30 |
| *** burgerk__ has joined #openstack-powervm | 20:30 | |
| *** burgerk_ has quit IRC | 20:34 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Add instance to many log events https://review.openstack.org/385144 | 20:37 |
| thorst | esberglu: is the CI having issues? See logs here: https://review.openstack.org/#/c/385132/ | 20:45 |
| efried | thorst, considered that, but a) the file is called powervm.py, so no collision there; and b) we're configuring the [powervm] conf section, so no collision there. | 20:45 |
| thorst | that looks like a VIOS thing | 20:45 |
| efried | But okay, next change set. | 20:45 |
| efried | thorst, you wanna fix your typo in the commit message? | 20:46 |
| thorst | efried: yeah, I am, but also doing a test locally quick | 20:46 |
| *** burgerk__ has quit IRC | 20:48 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Fix code for new pep8 rules https://review.openstack.org/385132 | 20:50 |
| thorst | efried: ^^ | 20:50 |
| thorst | I'll want to backport that to stable/mitaka and stable/newton once it passes. | 20:50 |
| efried | thorst, +2 | 20:51 |
| efried | thorst, are they backporting the hacking rule? | 20:51 |
| efried | - | 20:51 |
| efried | s | 20:51 |
| thorst | I guess I'll tox and find out | 20:51 |
| efried | I would be very surprised. | 20:52 |
| thorst | true true | 20:52 |
| efried | I mean, it's a good thing to fix those % in log messages, but not enough motivation for backporting. | 20:52 |
| efried | That's a great hacking rule, btw. Those are often tough to catch on a visual review. | 20:52 |
| *** svenkat has quit IRC | 20:58 | |
| thorst | efried: its back in mitaka | 21:02 |
| thorst | I mean newton | 21:02 |
| thorst | checking mitaka | 21:02 |
| efried | sheesh | 21:02 |
| thorst | efried: cherry picking back is going to be a joy. | 21:09 |
| openstackgerrit | Merged openstack/nova-powervm: Fix code for new pep8 rules https://review.openstack.org/385132 | 21:15 |
| efried | thorst, just to be sure, how did you check against newton? | 21:18 |
| thorst | I did a git checkout origin/stable/newton; tox -re pep8 | 21:19 |
| efried | Does that -r pick up the nova in branch stable/newton? | 21:22 |
| thorst | efried: shouldn't? | 21:23 |
| efried | Cause I don't see a cherry-pick of the change set that introduced the rule. | 21:23 |
| thorst | although...we may have a newton issue... | 21:23 |
| thorst | efried: yep...we've got a bug in our tox.ini instead | 21:24 |
| efried | I would be more sure of the result if I did: | 21:24 |
| thorst | let me abandon those | 21:24 |
| efried | cd .tox/pep8/src/nova | 21:24 |
| efried | git checkout -b newton origin/stable/newton | 21:24 |
| efried | and then ran the tox without -r. | 21:24 |
| thorst | our tox.ini needs to be updated to reflect stable/newton of nova... | 21:24 |
| efried | ah, k. | 21:24 |
| thorst | efried: I've got to run...not sure if you have time to make that update? | 21:24 |
| esberglu | thorst: Sorry I missed your CI question. Probably just due to me redeploying and not killing all the runs properly | 21:28 |
| *** thorst has quit IRC | 21:29 | |
| *** esberglu has quit IRC | 21:39 | |
| *** tblakes has quit IRC | 21:39 | |
| *** esberglu has joined #openstack-powervm | 21:40 | |
| *** esberglu has quit IRC | 21:42 | |
| *** seroyer has quit IRC | 21:58 | |
| *** tjakobs has quit IRC | 22:13 | |
| *** esberglu has joined #openstack-powervm | 22:15 | |
| *** esberglu has quit IRC | 22:20 | |
| *** apearson has quit IRC | 22:36 | |
| *** esberglu has joined #openstack-powervm | 22:39 | |
| *** tjakobs has joined #openstack-powervm | 22:44 | |
| *** tjakobs has quit IRC | 23:06 | |
| *** k0da has quit IRC | 23:07 | |
| *** tblakes has joined #openstack-powervm | 23:25 | |
| *** edmondsw has quit IRC | 23:30 | |
| *** seroyer has joined #openstack-powervm | 23:33 | |
| *** tblakes has quit IRC | 23:33 | |
| *** svenkat has joined #openstack-powervm | 23:54 | |
| *** seroyer has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!