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