14:01:19 <esberglu> #startmeeting powervm_driver_meeting 14:01:19 <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:20 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:01:22 <openstack> The meeting name has been set to 'powervm_driver_meeting' 14:01:44 <efried> \o 14:02:03 <esberglu> #topic In-tree Driver 14:02:06 <edmondsw> o/ 14:02:38 <esberglu> Just waiting for cores and updating as reviews come in 14:02:58 <edmondsw> we have until Friday to get everything in, as I understand it 14:03:19 <edmondsw> efried should we ping folks again? 14:03:58 <efried> I... guess? 14:04:09 <efried> Did gibi re-review the OVS patch? 14:04:25 <edmondsw> no. I wasn't sure if you'd already pinged him again 14:04:42 <edmondsw> I can ping folks, I just don't want to ping folks you've already pinged and be a nuisance 14:05:50 <efried> I hit up gibi just now 14:05:52 <edmondsw> tx 14:06:56 <edmondsw> and stephenfin, I see. I'll let you ping folks since you're already doing that. Thanks 14:07:21 <efried> Note that mriedem is out this week (and maybe next) 14:07:35 <edmondsw> yeah, I saw 14:08:39 <edmondsw> alright, we'll see how it goes 14:08:41 <efried> yuh 14:08:48 <edmondsw> next? 14:09:03 <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:18 <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:21 <edmondsw> pretty sure it's this week... next week is clients 14:09:45 <edmondsw> but... there is some hope if they slip this week, as efried said 14:09:54 <efried> Jan 25th is ff: https://wiki.openstack.org/wiki/Nova/Queens_Release_Schedule 14:10:04 <efried> This Thurs is non-client lib 14:10:29 <efried> Note that that's the openstack schedule. Nova usually follows it to within a few days, but sometimes not exactly. 14:10:34 <edmondsw> Jan 25 is the ff for clients, Jan 18 (yeah, Thurs, not Fri) is the ff for everything else 14:10:43 <efried> whoops, sorry 14:10:47 <efried> that *is* the nova schedule. 14:12:01 <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:25 <edmondsw> I just bumped the latter to keep up with g-r, so lost your +1s there 14:13:00 <efried> link pls 14:13:05 <efried> save me finding it. 14:13:13 <edmondsw> https://review.openstack.org/#/c/533779/ 14:13:30 <edmondsw> I think that's all IT? 14:13:45 <efried> oh, the g-r taskflow bump merged? 14:13:52 <esberglu> #topic Out-of-tree Driver 14:13:54 <edmondsw> efried yep 14:13:57 <efried> niiice 14:14:37 <esberglu> I've been working through some random backlog items 14:14:53 <efried> I strenuously object on moral grounds to the two-spaces-to-one thing. 14:15:27 <esberglu> efried: Doesn't matter to me, I just wanted to see if I could get the bash right 14:15:41 <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:56 <efried> bash? 14:16:07 <edmondsw> I assume he wrote a bash script to make the changes 14:16:13 <esberglu> yeah 14:16:18 <efried> oh, gotcha. 14:16:21 <edmondsw> and for what it's worth, that seems to have worked 14:16:57 <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:17:22 <efried> I was being (somewhat) facetious. 14:17:45 <efried> I do object to people getting uppity about one space, because they know not why they protest. 14:18:01 <edmondsw> lol... I hope I'm not getting uppity about it :) 14:18:11 <efried> And the reason doesn't apply to fixed-width. 14:18:24 <edmondsw> +1 14:18:40 <efried> I'll review the patch later. 14:19:02 <edmondsw> yeah, not a priority 14:19:13 <efried> Though tbh, I'm not sure it's worth muddying the git blame for five hundred lines. 14:19:25 <efried> But I'm not going to approve it without at least glancing at it. 14:19:56 <efried> Okay, we've already spent too much time on it; let's move on. 14:20:31 <edmondsw> I just saw there's a patch up for refactoring volume drivers 14:20:34 <edmondsw> I haven't looked yet 14:20:35 <edmondsw> https://review.openstack.org/#/c/530816/ 14:20:50 <edmondsw> will try to look at that today 14:21:05 <efried> on my list too. 14:21:51 <edmondsw> efried I wanted to ask what your plans are for 6121 14:22:05 <edmondsw> I gave some comments there a while back 14:22:17 <efried> Yeah. It hasn't been a priority. 14:22:21 <edmondsw> probably just sitting until we get past FF I assume 14:22:36 <efried> Doing what you request will be nontrivial and risky. 14:22:52 <efried> So if I'm going to do it, I know I'm going to need a decent time box. 14:23:03 <edmondsw> do you not think it needs doing? 14:23:30 <efried> If I didn't think there was merit to the idea, I would have just started arguing right away :) 14:23:38 <edmondsw> :) 14:23:45 <efried> I'm still not sure it's *worth* doing, but empirically you're right. 14:23:50 <openstackgerrit> Merged openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/533860 14:24:14 <edmondsw> bwarrum threw up a change for marking the encryption stuff expiremental, and I +1'd after one change 14:24:24 <edmondsw> I didn't +2 because I wanted to make sure you like it 14:24:39 <efried> Yes, I definitely need to look at that. 14:25:06 <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:10 <efried> to make sure nothing was missed. 14:25:18 <edmondsw> ++ 14:25:29 <efried> but again, need a time box for that. 14:25:48 <efried> I think I'll probably get it this morning, because I'm *way* ahead of Jay in terms of code-for-reviews. 14:26:22 <edmondsw> cool 14:26:37 <edmondsw> I think that's all I had for OOT 14:27:57 <esberglu> #topic Device Passthrough 14:28:20 <esberglu> Same status as last week I'm assuming 14:28:22 <efried> samo samo 14:28:23 <efried> yup 14:28:32 <efried> Though things are looking slippy 14:28:36 <efried> because Jay's been out. 14:29:23 <edmondsw> efried I saw him online yesterday I think? Is he back 100%? 14:29:35 <efried> He said 75%. But yeah, he's back. 14:29:47 <edmondsw> flu sounds nasty this year 14:30:24 <efried> yeah, and I didn't get a flu shot. I'm afeared. 14:30:40 * edmondsw crosses fingers 14:31:03 <edmondsw> I did have a thought on this 14:31:19 <edmondsw> triggered by the feature doc update that esberglu put up yesterday 14:31:29 <openstackgerrit> Merged openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/533906 14:31:43 <edmondsw> there is another section to that doc for HPC, and it includes a GPU attach section 14:32:21 <edmondsw> we should be working toward being able to include PowerVM as "complete" there 14:32:49 <edmondsw> so let's be thinking about that for Rocky 14:33:22 <edmondsw> all I had 14:33:46 <esberglu> #topic PowerVM CI 14:34:16 <esberglu> Starting to look into some stuff that I've been sitting on here 14:34:25 <openstackgerrit> Merged openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/533912 14:34:35 <esberglu> https://etherpad.openstack.org/p/powervm_ci_todos 14:34:53 <esberglu> That's pretty up to date, I can go into more detail here if you want 14:35:06 <esberglu> A lot of small cleanup 14:35:35 <esberglu> Then the next big change will be the pike undercloud upgrade started in change 6200 14:35:39 <edmondsw> esberglu will we be able to test OVS and SEA both in the same CI run? 14:36:40 <esberglu> edmondsw: We should be able to get 2 pipelines going and run both on each patch 14:36:47 <esberglu> But OVS uses WAY more resources 14:37:27 <edmondsw> k 14:37:57 <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:57 <edmondsw> I'll look over the TODO list in more detail post-meeting 14:39:14 <esberglu> #topic Open Discussion 14:39:16 <esberglu> Anything else? 14:40:29 <efried> Nothing from me 14:40:32 <edmondsw> probably not for today 14:41:10 <esberglu> #endmeeting