| *** edmondsw has joined #openstack-powervm | 00:13 | |
| *** edmondsw has quit IRC | 00:18 | |
| *** esberglu has joined #openstack-powervm | 00:30 | |
| *** esberglu has quit IRC | 00:45 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 00:48 | |
| *** AlexeyAbashkin has quit IRC | 00:52 | |
| *** edmondsw has joined #openstack-powervm | 01:00 | |
| *** jwcroppe has quit IRC | 01:08 | |
| *** jwcroppe has joined #openstack-powervm | 01:11 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 01:28 | |
| *** AlexeyAbashkin has quit IRC | 01:32 | |
| *** tjakobs_ has joined #openstack-powervm | 02:19 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 02:27 | |
| *** AlexeyAbashkin has quit IRC | 02:31 | |
| *** tonyb has quit IRC | 02:36 | |
| *** tonyb has joined #openstack-powervm | 02:37 | |
| *** tjakobs_ has quit IRC | 03:02 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 03:26 | |
| *** tjakobs_ has joined #openstack-powervm | 03:28 | |
| *** AlexeyAbashkin has quit IRC | 03:31 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 04:26 | |
| *** AlexeyAbashkin has quit IRC | 04:31 | |
| *** jwcroppe has quit IRC | 04:35 | |
| *** esberglu has joined #openstack-powervm | 05:09 | |
| *** esberglu has quit IRC | 05:09 | |
| *** edmondsw has quit IRC | 05:17 | |
| *** tjakobs_ has quit IRC | 05:29 | |
| *** jwcroppe has joined #openstack-powervm | 05:55 | |
| *** jwcroppe has quit IRC | 05:59 | |
| *** edmondsw has joined #openstack-powervm | 07:01 | |
| *** edmondsw has quit IRC | 07:06 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:34 | |
| *** edmondsw has joined #openstack-powervm | 08:50 | |
| *** edmondsw has quit IRC | 08:54 | |
| *** esberglu has joined #openstack-powervm | 09:04 | |
| *** esberglu has quit IRC | 09:09 | |
| *** esberglu has joined #openstack-powervm | 09:59 | |
| *** esberglu has quit IRC | 09:59 | |
| *** openstackgerrit has quit IRC | 10:33 | |
| *** edmondsw has joined #openstack-powervm | 10:38 | |
| *** edmondsw has quit IRC | 10:42 | |
| *** esberglu has joined #openstack-powervm | 10:52 | |
| *** esberglu has quit IRC | 10:57 | |
| *** edmondsw has joined #openstack-powervm | 12:09 | |
| *** esberglu has joined #openstack-powervm | 12:52 | |
| esberglu | #startmeeting powervm_driver_meeting | 13:00 |
|---|---|---|
| openstack | Meeting started Tue Oct 17 13:00:45 2017 UTC and is due to finish in 60 minutes. The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| *** openstack changes topic to " (Meeting topic: powervm_driver_meeting)" | 13:00 | |
| openstack | The meeting name has been set to 'powervm_driver_meeting' | 13:00 |
| edmondsw | o/ | 13:00 |
| efried | \o | 13:01 |
| esberglu | #topic In-Tree Driver | 13:01 |
| *** openstack changes topic to "In-Tree Driver (Meeting topic: powervm_driver_meeting)" | 13:01 | |
| esberglu | Not much motion here in the last week | 13:01 |
| esberglu | Config drive still up for review | 13:01 |
| esberglu | I spent some time at the end of last week trying to get and OVS devstack run to work | 13:02 |
| esberglu | Still working on that | 13:02 |
| edmondsw | how's that going? | 13:02 |
| esberglu | That's about it for me IT | 13:02 |
| edmondsw | stuck, or making progress? | 13:02 |
| esberglu | edmondsw: I got an OOT stack working with OVS, but haven't tried the equivalent IT yet | 13:02 |
| edmondsw | k, cool | 13:03 |
| esberglu | Figured I should be able to base most of it on that | 13:03 |
| edmondsw | I keep getting distracted when I go to review the config drive patch | 13:03 |
| edmondsw | efried once I review that, you think we can start asking for reviews in the community, or.. ? | 13:04 |
| edmondsw | should we have thorst review first? | 13:04 |
| efried | Nah, we should be able to ask sdague | 13:04 |
| edmondsw | k, I'll try to look at that today | 13:05 |
| esberglu | edmondsw: sweet | 13:05 |
| edmondsw | #action edmondsw to review IT config drive patch | 13:05 |
| edmondsw | that's probably all for IT then | 13:05 |
| esberglu | #topic Out-of-Tree Driver | 13:05 |
| *** openstack changes topic to "Out-of-Tree Driver (Meeting topic: powervm_driver_meeting)" | 13:06 | |
| esberglu | I spent a little time yesterday afternoon working on porting the host cpu stats to pypowervm | 13:06 |
| esberglu | Got the UT working, but the 1st try is failing sonar | 13:06 |
| efried | whee | 13:07 |
| esberglu | edmondsw: You said we need that by next week? | 13:07 |
| edmondsw | sounds like a good start, though. Can't ever expect sonar to work first time ;) | 13:07 |
| edmondsw | yeah | 13:07 |
| edmondsw | let's try to finish it this week if we can | 13:07 |
| esberglu | edmondsw: Yep I'll finish it this week to leave time for reviews | 13:08 |
| edmondsw | mdrabe efried should we talk about the DefaultStandardize change? | 13:09 |
| efried | mdrabe not here? | 13:09 |
| edmondsw | mdrabe you here? | 13:09 |
| edmondsw | oh nope... | 13:09 |
| efried | I can summarize the issue, if you like. | 13:09 |
| efried | The way lpar_builder is set up, it can't actually tell if it's creating a new LPAR or updating an existing one. | 13:10 |
| edmondsw | I've been following along, just wanted to discuss next steps | 13:10 |
| efried | So if you use DefaultStandardize, you'll always blat the values in its init with the defaults. | 13:10 |
| efried | Including max_io_slots | 13:10 |
| efried | So this change set is detecting when we have an existing LPAR and overwriting the DefaultStandardize values with the existing values from the wrapper. Which is stupid. But is actually the least evil AFAICT. | 13:11 |
| edmondsw | what I didn't quite follow was why there were two values we have to pick between which to keep... | 13:11 |
| edmondsw | one from the wrapper, and another from slot_mgr | 13:12 |
| edmondsw | can you explain that? | 13:12 |
| edmondsw | I think you guys agreed to always keep whichever of the 2 was higher | 13:13 |
| *** thorst has joined #openstack-powervm | 13:14 | |
| efried | It's possibly because this was a problem before, but we were in such a rush to implement the damned slot manager stuff, we just went with the first solution we came up with. | 13:14 |
| edmondsw | how could the slot_mgr have a value that's more correct about the current state of the VM than the lpar wrapper? | 13:15 |
| efried | I believe the slot manager figures out the highest slot being used by the source. | 13:15 |
| efried | Oh, because if the original LPAR was created with a non-default slot limit - which would have been possible before I introduced that bug - then you could have wound up using a slot higher than the default max (64). | 13:16 |
| efried | Then when you went to rebuild, it would rebuild the new LPAR with the default (64) and try to put a mapping on the >64 slot - kaboom. | 13:16 |
| efried | Oh, yeah, so it's not actually a horrible patchup solution. Because on a RR, we're creating the target LPAR afresh. | 13:17 |
| edmondsw | ok, so it's all about that issue with 1.1.7 | 13:17 |
| efried | So anyway, on a RR, we want to use whatever the slot manager said (which in fact is probably whatever was on the source LPAR, not just arbitrarily the highest); but on a resize we want to use whatever the original LPAR had. | 13:18 |
| efried | No, not entirely. | 13:18 |
| efried | The resize issue was still a bug, even before 1.1.7 | 13:18 |
| edmondsw | oh, true | 13:18 |
| efried | Try to resize an LPAR that started off with a nonstandard max_io_slots (or in fact any of those other fields DefaultStandardize defaults), and you'll end up with the defaults in those fields. | 13:18 |
| efried | I'm actually not completely sure there'll ever *be* a scenario where we'll have *both* a source LPAR and a slot manager. | 13:19 |
| efried | So in fact the "choose the highest" thing might be moot. For now. But better to do it and have it be a no-op than to have to track down THAT bug later on. | 13:20 |
| edmondsw | it sounded like you said instead of "choose highest" you're now thinking we should choose one or the other based on resize vs. RR | 13:21 |
| efried | No, actually. | 13:21 |
| efried | If both of those things are in play for some reason, we should choose the highest. | 13:21 |
| edmondsw | k | 13:22 |
| edmondsw | I think that makes sense | 13:22 |
| edmondsw | will want some nice comments about that in the code | 13:22 |
| edmondsw | explaining why | 13:22 |
| efried | Cause there's some semantic like "cold migrate resize" in play. | 13:22 |
| efried | or maybe it's "live resize with migration" | 13:23 |
| efried | Saw a bp go by about that. | 13:23 |
| edmondsw | ok, probably enough time on that | 13:23 |
| edmondsw | tx for the explanation | 13:23 |
| edmondsw | the only other thing I know of going on OOT is the iscsi work | 13:23 |
| efried | yuh, moving on. | 13:23 |
| edmondsw | we've been making progress there | 13:23 |
| edmondsw | tjakobs has been working to address comments | 13:24 |
| edmondsw | that's probably all for OOT | 13:24 |
| esberglu | #topic Device Passthrough | 13:24 |
| *** openstack changes topic to "Device Passthrough (Meeting topic: powervm_driver_meeting)" | 13:24 | |
| edmondsw | efried you're up | 13:25 |
| efried | Still waiting for jwcroppe to come back on requirements. | 13:25 |
| efried | I started trying to stack that new system, as time has allowed. | 13:26 |
| efried | Haven't succeeded yet, I don't think. | 13:26 |
| efried | But otherwise, nothing to report on the pre-Queens-hackup version. | 13:26 |
| efried | The Queens stuff is going fairly well, though. It's possible we could be able to do everything we need with generic RPs on a Queens base. | 13:27 |
| edmondsw | efried I didn't realize you were still waiting for input from jwcroppe. Are you sure he knows that? | 13:27 |
| efried | yeah | 13:27 |
| efried | You guys were gonna review my spec on granular resource request syntax, remember? | 13:27 |
| efried | https://review.openstack.org/#/c/510244/ | 13:27 |
| edmondsw | that's good news about queens | 13:27 |
| edmondsw | yeah, I remember... | 13:27 |
| esberglu | efried: I will today | 13:27 |
| edmondsw | #action edmondsw and esberglu to review https://review.openstack.org/#/c/510244/ | 13:28 |
| efried | Note that spec freeze is Thursday, so don't -1 me for typos. | 13:28 |
| efried | just for substantive stuff. | 13:28 |
| edmondsw | ack | 13:29 |
| esberglu | Yep Queens-1 thursday, forgot to say that during the drivers bit thanks | 13:29 |
| esberglu | efried: Anything else before we move on? | 13:30 |
| efried | Jay is on vacation | 13:30 |
| edmondsw | Jay is changing companies | 13:30 |
| efried | And I volunteered to babysit his NRP seres while he's gone. | 13:30 |
| efried | Yes | 13:30 |
| edmondsw | cool | 13:31 |
| edmondsw | alright... CI? | 13:32 |
| esberglu | #topic PowerVM CI | 13:32 |
| *** openstack changes topic to "PowerVM CI (Meeting topic: powervm_driver_meeting)" | 13:32 | |
| esberglu | Need to check with nvcastet today on that bug we've been seeing in CI | 13:32 |
| edmondsw | hsien should also be back now | 13:32 |
| esberglu | Other than that mostly just backlog items as able, nothing too exciting | 13:33 |
| esberglu | #topic Open Discussion | 13:34 |
| *** openstack changes topic to "Open Discussion (Meeting topic: powervm_driver_meeting)" | 13:34 | |
| edmondsw | nothing more from me | 13:34 |
| esberglu | neither | 13:35 |
| esberglu | #endmeeting | 13:35 |
| *** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 13:35 | |
| openstack | Meeting ended Tue Oct 17 13:35:55 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:35 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-10-17-13.00.html | 13:35 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-10-17-13.00.txt | 13:36 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-10-17-13.00.log.html | 13:36 |
| *** esberglu has quit IRC | 13:54 | |
| *** esberglu has joined #openstack-powervm | 13:54 | |
| *** esberglu has quit IRC | 13:59 | |
| *** jwcroppe has joined #openstack-powervm | 14:06 | |
| *** esberglu has joined #openstack-powervm | 14:09 | |
| *** tjakobs_ has joined #openstack-powervm | 14:15 | |
| *** openstackgerrit has joined #openstack-powervm | 14:41 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: ci check https://review.openstack.org/328315 | 14:41 |
| *** AlexeyAbashkin has quit IRC | 15:30 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 15:30 | |
| *** AlexeyAbashkin has quit IRC | 15:41 | |
| *** jwcroppe_ has joined #openstack-powervm | 15:47 | |
| *** jwcroppe has quit IRC | 15:49 | |
| *** jwcroppe_ has quit IRC | 17:13 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 17:22 | |
| *** AlexeyAbashkin has quit IRC | 17:24 | |
| *** cjvolzka has joined #openstack-powervm | 19:06 | |
| *** jwcroppe has joined #openstack-powervm | 19:21 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 19:40 | |
| *** AlexeyAbashkin has quit IRC | 19:44 | |
| *** jwcroppe has quit IRC | 19:45 | |
| *** jwcroppe has joined #openstack-powervm | 20:02 | |
| efried | esberglu edmondsw Some jackass is making a breaking change to the spawn method signature. We'll need to update our OOT driver accordingly. | 20:07 |
| efried | Either one of y'all want to take that on, or want me to do it? | 20:08 |
| edmondsw | lovely | 20:08 |
| edmondsw | efried if you have the time... I'm neck deep | 20:08 |
| efried | Okay. I'm pretty familiar with the change, so I guess I can do it. FYI: https://review.openstack.org/511879 | 20:09 |
| efried | esberglu If we want CI results before ^ merges, we'll have to patch it in somehow. Depends-On doesn't work for us from OOT, does it? | 20:10 |
| edmondsw | I don't think so | 20:10 |
| esberglu | efried: No it doesn't. I can do a manual run | 20:13 |
| efried | esberglu ight, stand by. | 20:13 |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Add allocations to spawn signature https://review.openstack.org/512814 | 20:19 |
| efried | esberglu ^ + https://review.openstack.org/511879 | 20:19 |
| esberglu | efried: ack | 20:19 |
| efried | esberglu Somehow our OOT CI passed at PS5 of the above. Can't imagine how. | 20:21 |
| efried | oh, we're not running OOT CI on nova changes?? | 20:21 |
| efried | esberglu wait, I'm confused. | 20:21 |
| esberglu | efried: I know what happened | 20:21 |
| efried | okay. FYI, I've also seen this pop up more than once: http://184.172.12.213/79/511879/9/check/nova-in-tree-pvm/81b6c39/powervm_os_ci.html | 20:22 |
| *** AlexeyAbashkin has joined #openstack-powervm | 20:22 | |
| efried | esberglu Let me know when I can powervm:recheck the nova change. | 20:23 |
| efried | esberglu Oh, by the way, if we want to patch in https://review.openstack.org/#/c/493654/ (Glance) and https://review.openstack.org/492651 (devstack) we can start using uwsgi | 20:24 |
| efried | I tested it locally and it seems to work fine. | 20:25 |
| esberglu | efried: We skip OOT CI on nova changes with powervm in any of the filepaths | 20:25 |
| efried | esberglu Ahhhhh. Well, in this case, that's wrong :) | 20:25 |
| efried | Or... maybe not, actually. Cause this will in fact break OOT. | 20:25 |
| efried | Okay, I'll recheck to get the in-tree one validated. | 20:26 |
| efried | esberglu Unless there's some know permanent issue with that multiple-networks thing | 20:26 |
| esberglu | efried: Nah that's an issue that hits the IT driver very infrequently | 20:27 |
| *** AlexeyAbashkin has quit IRC | 20:27 | |
| efried | ight, rechecking. | 20:27 |
| esberglu | efried: Do you remember why we were skipping OOT CI on IT patches with powervm in them? | 20:35 |
| *** jwcroppe has quit IRC | 20:35 | |
| esberglu | I'm struggling to recall and the commit message wasn't very helpful | 20:35 |
| efried | Probably because we assumed they were going to be adds to the in-tree driver, and therefore the OOT driver would override them anyway and be redundant, so we probably did it to save CI resources. | 20:36 |
| efried | esberglu If we're not worried about CI resources, it's probably not a terrible idea to take that condition away. | 20:37 |
| esberglu | efried: It will make the CI jobs SO much more readable to get rid of that condition | 20:37 |
| esberglu | And it shouldn't affect overall run numbers really | 20:38 |
| efried | esberglu Is there some kind of condition going the other way too? Like, we skip the IT runs for all OOT patches, or something? | 20:38 |
| esberglu | efried: Nope | 20:38 |
| efried | Hm. Then there was probably a much better reason we were skipping OOT runs on powervm-containing IT patches. | 20:39 |
| efried | But I can't remember what it would have been. | 20:39 |
| openstackgerrit | Merged openstack/nova-powervm master: Support PPT ratio capable to PPT ratio incapable host rebuild https://review.openstack.org/508236 | 20:39 |
| *** edmondsw has quit IRC | 21:16 | |
| *** edmondsw_ has joined #openstack-powervm | 21:18 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: WIP: Move host cpu stats to pypowervm https://review.openstack.org/512864 | 21:19 |
| *** edmondsw_ has quit IRC | 21:22 | |
| *** thorst has quit IRC | 21:25 | |
| *** thorst has joined #openstack-powervm | 21:25 | |
| *** thorst has quit IRC | 21:30 | |
| *** thorst has joined #openstack-powervm | 21:50 | |
| *** edmondsw has joined #openstack-powervm | 21:51 | |
| *** thorst has quit IRC | 21:54 | |
| *** edmondsw has quit IRC | 21:55 | |
| *** esberglu has quit IRC | 21:55 | |
| *** esberglu has joined #openstack-powervm | 21:56 | |
| *** esberglu has quit IRC | 22:00 | |
| *** tjakobs_ has quit IRC | 22:12 | |
| *** esberglu has joined #openstack-powervm | 23:31 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!