| *** k0da has joined #openstack-powervm | 00:08 | |
| *** k0da has quit IRC | 00:28 | |
| *** csky has quit IRC | 00:54 | |
| *** csky has joined #openstack-powervm | 01:25 | |
| *** csky has quit IRC | 02:24 | |
| *** csky has joined #openstack-powervm | 02:26 | |
| *** edmondsw has quit IRC | 02:34 | |
| *** csky has quit IRC | 02:43 | |
| *** csky has joined #openstack-powervm | 03:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/533860 | 03:18 |
|---|---|---|
| *** csky has quit IRC | 03:34 | |
| *** edmondsw has joined #openstack-powervm | 03:47 | |
| *** edmondsw has quit IRC | 03:52 | |
| *** csky has joined #openstack-powervm | 04:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/533906 | 04:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/533912 | 04:20 |
| *** chhavi has joined #openstack-powervm | 04:31 | |
| *** chhagarw has joined #openstack-powervm | 04:31 | |
| *** csky has quit IRC | 05:39 | |
| *** csky has joined #openstack-powervm | 05:46 | |
| *** csky has quit IRC | 06:00 | |
| *** csky has joined #openstack-powervm | 06:27 | |
| *** csky has quit IRC | 07:03 | |
| *** edmondsw has joined #openstack-powervm | 07:24 | |
| *** edmondsw has quit IRC | 07:28 | |
| *** csky has joined #openstack-powervm | 07:29 | |
| *** csky has quit IRC | 08:08 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 08:15 | |
| *** csky has joined #openstack-powervm | 08:35 | |
| *** csky has quit IRC | 08:59 | |
| *** edmondsw has joined #openstack-powervm | 09:12 | |
| *** k0da has joined #openstack-powervm | 09:14 | |
| *** edmondsw has quit IRC | 09:17 | |
| *** csky has joined #openstack-powervm | 09:26 | |
| *** csky has quit IRC | 09:51 | |
| *** csky has joined #openstack-powervm | 10:18 | |
| *** AlexeyAbashkin has quit IRC | 10:23 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 10:26 | |
| *** chhavi has quit IRC | 10:28 | |
| *** chhagarw has quit IRC | 10:29 | |
| *** csky has quit IRC | 10:38 | |
| *** edmondsw has joined #openstack-powervm | 11:00 | |
| *** csky has joined #openstack-powervm | 11:04 | |
| *** edmondsw has quit IRC | 11:05 | |
| *** csky has quit IRC | 11:30 | |
| *** edmondsw has joined #openstack-powervm | 12:03 | |
| *** csky has joined #openstack-powervm | 12:48 | |
| *** csky has quit IRC | 13:21 | |
| *** esberglu has joined #openstack-powervm | 13:59 | |
| edmondsw | esberglu mtg? | 14:01 |
| esberglu | #startmeeting powervm_driver_meeting | 14:01 |
| openstack | Meeting started Tue Jan 16 14:01:19 2018 UTC and is due to finish in 60 minutes. The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| *** openstack changes topic to " (Meeting topic: powervm_driver_meeting)" | 14:01 | |
| openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:01 |
| *** chhagarw has joined #openstack-powervm | 14:01 | |
| *** chhavi has joined #openstack-powervm | 14:01 | |
| efried | \o | 14:01 |
| esberglu | #topic In-tree Driver | 14:02 |
| *** openstack changes topic to "In-tree Driver (Meeting topic: powervm_driver_meeting)" | 14:02 | |
| edmondsw | o/ | 14:02 |
| esberglu | Just waiting for cores and updating as reviews come in | 14:02 |
| edmondsw | we have until Friday to get everything in, as I understand it | 14:02 |
| *** csky has joined #openstack-powervm | 14:03 | |
| edmondsw | efried should we ping folks again? | 14:03 |
| efried | I... guess? | 14:03 |
| efried | Did gibi re-review the OVS patch? | 14:04 |
| edmondsw | no. I wasn't sure if you'd already pinged him again | 14:04 |
| edmondsw | I can ping folks, I just don't want to ping folks you've already pinged and be a nuisance | 14:04 |
| efried | I hit up gibi just now | 14:05 |
| edmondsw | tx | 14:05 |
| edmondsw | and stephenfin, I see. I'll let you ping folks since you're already doing that. Thanks | 14:06 |
| efried | Note that mriedem is out this week (and maybe next) | 14:07 |
| edmondsw | yeah, I saw | 14:07 |
| edmondsw | alright, we'll see how it goes | 14:08 |
| efried | yuh | 14:08 |
| edmondsw | next? | 14:08 |
| esberglu | edmondsw: Are you sure it has to be this week? I thought this week was the freeze for stuff like oslo, os-vif, etc. | 14:09 |
| efried | Having patches "ready" is going to make it more likely they'll still be allowed to merge after the actual freeze date. | 14:09 |
| edmondsw | pretty sure it's this week... next week is clients | 14:09 |
| edmondsw | but... there is some hope if they slip this week, as efried said | 14:09 |
| efried | Jan 25th is ff: https://wiki.openstack.org/wiki/Nova/Queens_Release_Schedule | 14:09 |
| efried | This Thurs is non-client lib | 14:10 |
| efried | Note that that's the openstack schedule. Nova usually follows it to within a few days, but sometimes not exactly. | 14:10 |
| edmondsw | Jan 25 is the ff for clients, Jan 18 (yeah, Thurs, not Fri) is the ff for everything else | 14:10 |
| efried | whoops, sorry | 14:10 |
| efried | that *is* the nova schedule. | 14:10 |
| edmondsw | oh, one more thing IT... we have esberglu's patch for the feature docs adding powervm and mine to add taskflow as a requirement | 14:12 |
| edmondsw | I just bumped the latter to keep up with g-r, so lost your +1s there | 14:12 |
| efried | link pls | 14:13 |
| efried | save me finding it. | 14:13 |
| edmondsw | https://review.openstack.org/#/c/533779/ | 14:13 |
| edmondsw | I think that's all IT? | 14:13 |
| efried | oh, the g-r taskflow bump merged? | 14:13 |
| esberglu | #topic Out-of-tree Driver | 14:13 |
| *** openstack changes topic to "Out-of-tree Driver (Meeting topic: powervm_driver_meeting)" | 14:13 | |
| edmondsw | efried yep | 14:13 |
| efried | niiice | 14:13 |
| esberglu | I've been working through some random backlog items | 14:14 |
| efried | I strenuously object on moral grounds to the two-spaces-to-one thing. | 14:14 |
| esberglu | efried: Doesn't matter to me, I just wanted to see if I could get the bash right | 14:15 |
| edmondsw | I was gonna say we probably didn't NEED to do that... I like consistency, but it's not effort we need to spend | 14:15 |
| efried | bash? | 14:15 |
| edmondsw | I assume he wrote a bash script to make the changes | 14:16 |
| esberglu | yeah | 14:16 |
| efried | oh, gotcha. | 14:16 |
| edmondsw | and for what it's worth, that seems to have worked | 14:16 |
| edmondsw | so efried if you object to merging that, just -1 or -2 and we can abandon it. I won't push it. I was just going to say merge it if you didn't object, since the work is done and consistency | 14:16 |
| efried | I was being (somewhat) facetious. | 14:17 |
| efried | I do object to people getting uppity about one space, because they know not why they protest. | 14:17 |
| edmondsw | lol... I hope I'm not getting uppity about it :) | 14:18 |
| efried | And the reason doesn't apply to fixed-width. | 14:18 |
| edmondsw | +1 | 14:18 |
| efried | I'll review the patch later. | 14:18 |
| edmondsw | yeah, not a priority | 14:19 |
| efried | Though tbh, I'm not sure it's worth muddying the git blame for five hundred lines. | 14:19 |
| efried | But I'm not going to approve it without at least glancing at it. | 14:19 |
| efried | Okay, we've already spent too much time on it; let's move on. | 14:19 |
| edmondsw | I just saw there's a patch up for refactoring volume drivers | 14:20 |
| edmondsw | I haven't looked yet | 14:20 |
| edmondsw | https://review.openstack.org/#/c/530816/ | 14:20 |
| edmondsw | will try to look at that today | 14:20 |
| efried | on my list too. | 14:21 |
| edmondsw | efried I wanted to ask what your plans are for 6121 | 14:21 |
| edmondsw | I gave some comments there a while back | 14:22 |
| efried | Yeah. It hasn't been a priority. | 14:22 |
| edmondsw | probably just sitting until we get past FF I assume | 14:22 |
| efried | Doing what you request will be nontrivial and risky. | 14:22 |
| efried | So if I'm going to do it, I know I'm going to need a decent time box. | 14:22 |
| edmondsw | do you not think it needs doing? | 14:23 |
| efried | If I didn't think there was merit to the idea, I would have just started arguing right away :) | 14:23 |
| edmondsw | :) | 14:23 |
| efried | I'm still not sure it's *worth* doing, but empirically you're right. | 14:23 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/533860 | 14:23 |
| edmondsw | bwarrum threw up a change for marking the encryption stuff expiremental, and I +1'd after one change | 14:24 |
| edmondsw | I didn't +2 because I wanted to make sure you like it | 14:24 |
| efried | Yes, I definitely need to look at that. | 14:24 |
| efried | My plan was actually to do a diff against HEAD~2 - i.e. look at the encryption stuff as if it had been put in that way. | 14:25 |
| efried | to make sure nothing was missed. | 14:25 |
| edmondsw | ++ | 14:25 |
| efried | but again, need a time box for that. | 14:25 |
| efried | I think I'll probably get it this morning, because I'm *way* ahead of Jay in terms of code-for-reviews. | 14:25 |
| edmondsw | cool | 14:26 |
| edmondsw | I think that's all I had for OOT | 14:26 |
| esberglu | #topic Device Passthrough | 14:27 |
| *** openstack changes topic to "Device Passthrough (Meeting topic: powervm_driver_meeting)" | 14:27 | |
| esberglu | Same status as last week I'm assuming | 14:28 |
| efried | samo samo | 14:28 |
| efried | yup | 14:28 |
| efried | Though things are looking slippy | 14:28 |
| efried | because Jay's been out. | 14:28 |
| edmondsw | efried I saw him online yesterday I think? Is he back 100%? | 14:29 |
| efried | He said 75%. But yeah, he's back. | 14:29 |
| edmondsw | flu sounds nasty this year | 14:29 |
| efried | yeah, and I didn't get a flu shot. I'm afeared. | 14:30 |
| * edmondsw crosses fingers | 14:30 | |
| edmondsw | I did have a thought on this | 14:31 |
| edmondsw | triggered by the feature doc update that esberglu put up yesterday | 14:31 |
| openstackgerrit | Merged openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/533906 | 14:31 |
| edmondsw | there is another section to that doc for HPC, and it includes a GPU attach section | 14:31 |
| edmondsw | we should be working toward being able to include PowerVM as "complete" there | 14:32 |
| edmondsw | so let's be thinking about that for Rocky | 14:32 |
| edmondsw | all I had | 14:33 |
| esberglu | #topic PowerVM CI | 14:33 |
| *** openstack changes topic to "PowerVM CI (Meeting topic: powervm_driver_meeting)" | 14:33 | |
| esberglu | Starting to look into some stuff that I've been sitting on here | 14:34 |
| openstackgerrit | Merged openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/533912 | 14:34 |
| esberglu | https://etherpad.openstack.org/p/powervm_ci_todos | 14:34 |
| esberglu | That's pretty up to date, I can go into more detail here if you want | 14:34 |
| esberglu | A lot of small cleanup | 14:35 |
| esberglu | Then the next big change will be the pike undercloud upgrade started in change 6200 | 14:35 |
| edmondsw | esberglu will we be able to test OVS and SEA both in the same CI run? | 14:35 |
| esberglu | edmondsw: We should be able to get 2 pipelines going and run both on each patch | 14:36 |
| esberglu | But OVS uses WAY more resources | 14:36 |
| edmondsw | k | 14:37 |
| esberglu | I stole my staging CI systems to test OVS and vSCSI. Once I get those back I can start messing with it | 14:37 |
| edmondsw | I'll look over the TODO list in more detail post-meeting | 14:37 |
| esberglu | #topic Open Discussion | 14:39 |
| *** openstack changes topic to "Open Discussion (Meeting topic: powervm_driver_meeting)" | 14:39 | |
| esberglu | Anything else? | 14:39 |
| efried | Nothing from me | 14:40 |
| edmondsw | probably not for today | 14:40 |
| esberglu | #endmeeting | 14:41 |
| *** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 14:41 | |
| openstack | Meeting ended Tue Jan 16 14:41:10 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:41 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-16-14.01.html | 14:41 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-16-14.01.txt | 14:41 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-16-14.01.log.html | 14:41 |
| edmondsw | esberglu efried can one of you +W on 6244? | 14:49 |
| edmondsw | or I guess I could just do that | 14:49 |
| edmondsw | tx | 14:49 |
| efried | Yeah, but done | 14:49 |
| edmondsw | efried to answer your question... I don't think OpenStack was trying to get networkx 2.0 support in for Queens, so I don't know if we have to spin a release ASAP, but I'll check with prometheanfire | 14:51 |
| efried | sounds good | 14:51 |
| edmondsw | esberglu I added you to https://review.openstack.org/#/c/530816/ | 15:22 |
| edmondsw | we had some conversations about this kind of thing when you were working on vSCSI for IT | 15:22 |
| edmondsw | please take a look and see how this aligns with those conversations | 15:22 |
| *** apearson has joined #openstack-powervm | 15:23 | |
| edmondsw | and what you think | 15:23 |
| esberglu | edmondsw: ack | 15:27 |
| *** esberglu has quit IRC | 15:32 | |
| *** esberglu has joined #openstack-powervm | 15:48 | |
| esberglu | efried: edmondsw: Single test skip list update in 6246 when you get a chance | 16:06 |
| esberglu | edmondsw: OVS is +2 from gibi | 16:17 |
| edmondsw | esberglu woot! | 16:18 |
| *** chhagarw has quit IRC | 16:24 | |
| *** chhavi has quit IRC | 16:24 | |
| *** k0da has quit IRC | 16:50 | |
| *** AlexeyAbashkin has quit IRC | 17:11 | |
| *** openstackgerrit has quit IRC | 17:33 | |
| esberglu | efried: edmondsw: Thoughts on https://review.openstack.org/#/c/422512/39/nova/virt/powervm/vif.py@105 ? | 17:44 |
| edmondsw | no, I don't see how neutron could use this | 17:49 |
| edmondsw | they only care about networking. They don't care about VMs | 17:49 |
| edmondsw | it's nova's job to tie VMs and networks together | 17:49 |
| edmondsw | unless I'm missing something... | 17:49 |
| efried | How about a response that covers our bases | 17:50 |
| efried | Like, "Pretty sure not, but if it did, we would want to pull it out in a future series" | 17:50 |
| esberglu | efried: What about the nova-net comment? Go along with gibi's comment and say we will remove all nova-net stuff and only claim neutron support in a future change? | 17:54 |
| efried | sure, sounds good to me. | 17:54 |
| edmondsw | I asked thorst if the OOT driver worked with nova-network and he said no | 18:01 |
| edmondsw | said they tried making that work a while back, hence the comments | 18:21 |
| esberglu | edmondsw: Think I should go ahead and rip it out? I don't want to lose the reviews, but I left a comment saying it's not supported, so I suspect I will have to respin anyways | 18:36 |
| esberglu | Or just wait and see what they say | 18:36 |
| edmondsw | yeah, I'm not sure. Maybe ask stephenfin on irc | 18:38 |
| edmondsw | if he does want you to respin, he can suggest where we should say that only neutron is supported | 18:38 |
| edmondsw | but I'd rather just leave it if he's ok with that | 18:38 |
| *** AlexeyAbashkin has joined #openstack-powervm | 18:43 | |
| efried | esberglu Know about git restack? | 18:45 |
| esberglu | efried: Nope | 18:45 |
| efried | pip install git-restack | 18:45 |
| efried | Then from the top of your series, run git restack | 18:46 |
| efried | Change 'pick' to 'edit' for the patch(es) you want to edit, save and quit the editor. | 18:46 |
| efried | When done editing, git rebase --continue | 18:46 |
| efried | Then git review will submit the whole series at once. | 18:46 |
| *** AlexeyAbashkin has quit IRC | 18:48 | |
| efried | So in this case if you're just editing the one patch, git restack will automatically do the rest of the rebasing for you. | 18:49 |
| efried | If it gets stuck, it'll stop and tell you so; you resolve conflicts just like you would any merge conflict and then git rebase --continue. | 18:49 |
| edmondsw | that's pretty cool | 18:50 |
| edmondsw | esberglu if you're gonna respin, better pull out not only comments re: nova-net but also the code for it | 18:50 |
| esberglu | edmondsw: Yep | 18:50 |
| edmondsw | and then better retest | 18:50 |
| edmondsw | esberglu re: 6246... how is that update method run by that test? It doesn't look like it should be from what I've found | 18:54 |
| edmondsw | did you have a stacktrace? I'm curious now | 18:54 |
| edmondsw | and wondering if rename is something that we need to add support for. I thought that would just work without any driver involvement (just rename in the nova db) | 18:55 |
| edmondsw | I thought the references to rename in our driver were just for renaming things on the backend during resize/migration | 18:56 |
| efried | We rename the lpar, nah? | 18:56 |
| edmondsw | do we? | 18:56 |
| esberglu | edmondsw: Let me finish this and then I can dig up my notes from last week. I could be mistaken | 18:56 |
| efried | But yeah, I thought our LparBuilder crap took care of all that. | 18:56 |
| efried | A little surprised if it doesn't. | 18:56 |
| efried | or if we left any of that out. | 18:56 |
| efried | I can't imagine we wanted to go through the pain of dissecting LparBuilder just so we could, what, reduce the LOC count or whatever? | 18:57 |
| *** efried is now known as efried_nomnom | 18:57 | |
| edmondsw | efried_nomnom LparBuilder is in pypowervm, so same for IT and OOT | 18:58 |
| efried_nomnom | VMBuilder, then | 18:59 |
| esberglu | efried_nomnom: Man git-restack is smooth | 18:59 |
| esberglu | Thanks for the tip | 18:59 |
| edmondsw | efried_nomnom ah, yeah | 19:00 |
| efried_nomnom | esberglu For sure. Still gotta be a little careful sometimes - mainly don't commit -a when you shouldn't - but it makes dealing with tall series WAY easier. | 19:00 |
| efried_nomnom | esberglu viz: https://review.openstack.org/#/c/533244/ -- 12 tall :) | 19:01 |
| esberglu | edmondsw: You're right about that test | 19:36 |
| edmondsw | esberglu so do we need to revert that patch? | 19:41 |
| edmondsw | is there another change we need instead? | 19:41 |
| esberglu | edmondsw: Yeah. And figure out what's actually happening | 19:41 |
| edmondsw | cool | 19:41 |
| edmondsw | sorry | 19:41 |
| edmondsw | efried ^ | 19:41 |
| esberglu | 6247 | 19:43 |
| esberglu | edmondsw: ^ | 19:43 |
| edmondsw | +2 | 19:46 |
| edmondsw | esberglu you might want to figure it out before merging the revert | 19:46 |
| edmondsw | just to keep things working | 19:46 |
| *** k0da has joined #openstack-powervm | 19:55 | |
| esberglu | edmondsw: It seems like this should be failing every time. The server name gets updated to non-ascii value, which causes the delete to fail | 20:14 |
| esberglu | But it always is the same non-ascii value | 20:15 |
| esberglu | Comparing successful and failing but not seeing what is different yet | 20:15 |
| edmondsw | esberglu where are the logs? | 20:17 |
| esberglu | Failure: http://184.172.12.213/86/521686/9/check/nova-in-tree-pvm/de0247e/logs/n-cpu.txt.gz?#_Jan_11_08_15_10_325954 | 20:17 |
| esberglu | Success: http://184.172.12.213/35/534335/1/check/nova-in-tree-pvm/effe2aa/logs/n-cpu.txt.gz#_Jan_16_09_33_47_429249 | 20:18 |
| esberglu | You can see the name gets changed to non-ascii on the success, with the terminate going through shortly after | 20:18 |
| edmondsw | could it be pulling different versions of taskflow and/or networkx ? | 20:21 |
| edmondsw | between success and failure | 20:21 |
| edmondsw | not sure where to look for that here | 20:21 |
| edmondsw | esberglu ^ | 20:22 |
| edmondsw | esberglu failure has taskflow 3.1.0 and success has 3.0.1, but same version of networkx (1.11) | 20:24 |
| esberglu | edmondsw: Where are you seeing that? | 20:24 |
| edmondsw | logs/stack.sh.txt.gz | 20:24 |
| edmondsw | esberglu because the error comes from networkx, called by taskflow: http://184.172.12.213/86/521686/9/check/nova-in-tree-pvm/de0247e/logs/n-cpu.txt.gz?#_Jan_11_08_15_10_337046 | 20:26 |
| esberglu | edmondsw: I'm seeing 3.1.0 for both? | 20:27 |
| edmondsw | esberglu oh, my bad | 20:28 |
| edmondsw | yeah, 3.1.0 for both | 20:28 |
| edmondsw | hmm | 20:28 |
| edmondsw | there goes that theory | 20:28 |
| esberglu | edmondsw: efried_nomnom: OVS is ready for new round of +1s | 20:40 |
| *** efried_nomnom is now known as efried | 20:42 | |
| edmondsw | esberglu gonna have a couple small comments | 20:43 |
| edmondsw | still looking | 20:43 |
| edmondsw | esberglu ok done | 20:44 |
| edmondsw | +1 | 20:47 |
| esberglu | edmondsw: Done | 20:47 |
| esberglu | tx | 20:47 |
| *** esberglu has quit IRC | 21:37 | |
| *** esberglu has joined #openstack-powervm | 21:50 | |
| *** chhagarw has joined #openstack-powervm | 22:20 | |
| *** chhavi has joined #openstack-powervm | 22:20 | |
| *** chhagarw has quit IRC | 22:24 | |
| *** chhavi has quit IRC | 22:24 | |
| edmondsw | esberglu did you see gibi's suggestion of adding a patch with a release note on powervm additions to the end of the series? | 22:49 |
| edmondsw | I liked that idea... | 22:50 |
| *** csky has quit IRC | 22:54 | |
| *** csky has joined #openstack-powervm | 22:55 | |
| *** edmondsw has quit IRC | 22:55 | |
| *** edmondsw has joined #openstack-powervm | 22:56 | |
| esberglu | edmondsw: Yeah will do | 22:57 |
| *** csky has quit IRC | 22:59 | |
| *** edmondsw has quit IRC | 23:00 | |
| *** csky has joined #openstack-powervm | 23:25 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!