*** edmondsw has joined #openstack-powervm | 03:18 | |
*** edmondsw has quit IRC | 03:22 | |
*** edmondsw has joined #openstack-powervm | 05:06 | |
*** edmondsw has quit IRC | 05:10 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/552309 | 06:48 |
---|---|---|
*** edmondsw has joined #openstack-powervm | 06:54 | |
*** edmondsw has quit IRC | 06:59 | |
openstackgerrit | ShangXiao proposed openstack/nova-powervm master: Update the old links in docs https://review.openstack.org/552379 | 07:26 |
*** AlexeyAbashkin has joined #openstack-powervm | 07:50 | |
*** edmondsw has joined #openstack-powervm | 08:42 | |
*** edmondsw has quit IRC | 08:46 | |
*** edmondsw has joined #openstack-powervm | 10:30 | |
*** edmondsw has quit IRC | 10:35 | |
*** edmondsw has joined #openstack-powervm | 11:45 | |
*** apearson_ has joined #openstack-powervm | 12:52 | |
edmondsw | tonyb did I read your comment about the meeting proposal correctly that there is a move *away* from using openstack-meeting channels? | 13:28 |
edmondsw | i.e. should I repropose for openstack-meeting-3 or just continue to have the meeting here like we have been? | 13:29 |
*** tblakes has joined #openstack-powervm | 13:30 | |
edmondsw | tonyb I was trying to follow what it says in the project requirements about "moving to official OpenStack meeting channels once the project’s application is accepted" | 13:32 |
edmondsw | https://governance.openstack.org/tc/reference/new-projects-requirements.html | 13:32 |
*** esberglu has joined #openstack-powervm | 13:53 | |
edmondsw | #startmeeting PowerVM Driver Meeting | 14:00 |
openstack | Meeting started Tue Mar 13 14:00:40 2018 UTC and is due to finish in 60 minutes. The chair is edmondsw. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
*** openstack changes topic to " (Meeting topic: PowerVM Driver Meeting)" | 14:00 | |
openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:00 |
edmondsw | #topic In-Tree Driver | 14:01 |
*** openstack changes topic to "In-Tree Driver (Meeting topic: PowerVM Driver Meeting)" | 14:01 | |
esberglu | \o | 14:01 |
edmondsw | efried? | 14:01 |
efried | Sorry, chaos | 14:01 |
edmondsw | np | 14:01 |
edmondsw | https://etherpad.openstack.org/p/powervm-in-tree-todos | 14:02 |
edmondsw | wanted to talk about what we're doing with the blurb at the top of that | 14:02 |
edmondsw | efried you probably have the best idea of what we should do there | 14:03 |
edmondsw | but if you're still in chaos we can come back to that | 14:03 |
efried | Sorry, guys, any way we can pick this up in 10 minutes? | 14:03 |
efried | otp and at the mercy of the other end | 14:03 |
edmondsw | fine with me | 14:03 |
esberglu | np for me | 14:03 |
efried | I'm back, sorry about that. | 14:16 |
efried | We're talking about the InstanceInfo thing? | 14:16 |
edmondsw | esberglu resuming | 14:16 |
esberglu | ack | 14:16 |
edmondsw | efried yeah... I don't think we have anything more to do there do we? | 14:16 |
efried | Did we flush those changes back to OOT? | 14:17 |
edmondsw | good question | 14:17 |
edmondsw | #action edmonds check whether we need InstanceInfo changes OOT | 14:17 |
edmondsw | I can move to the OOT todo list if need be | 14:17 |
edmondsw | as for Rocky... | 14:18 |
efried | Looks like we did. So yeah, we can remove that item. | 14:18 |
edmondsw | efried great, tx | 14:18 |
efried | def get_vm_info(adapter, instance): | 14:19 |
efried | """Get the InstanceInfo for an instance. | 14:19 |
efried | :param adapter: The pypowervm.adapter.Adapter for the PowerVM REST API. | 14:19 |
efried | :param instance: nova.objects.instance.Instance object | 14:19 |
efried | :returns: An InstanceInfo object. | 14:19 |
efried | """ | 14:19 |
efried | pvm_uuid = get_pvm_uuid(instance) | 14:19 |
efried | pvm_state = get_vm_qp(adapter, pvm_uuid, 'PartitionState') | 14:19 |
efried | nova_state = _translate_vm_state(pvm_state) | 14:19 |
efried | return hardware.InstanceInfo(nova_state) | 14:19 |
edmondsw | I added a couple reviews to https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | 14:19 |
edmondsw | including the spec for virt integration efforts in Rocky | 14:19 |
edmondsw | I'm hoping we can get that approved soon... | 14:19 |
esberglu | I added network hotplug and snapshot as ready for subteam review there | 14:20 |
edmondsw | efried do you have a feel for how long I should wait before pinging melwitt? | 14:20 |
edmondsw | esberglu tx... I think that'll be a good place for us to keep track of things | 14:20 |
edmondsw | and have visibility in the larger nova team to what we're doing | 14:21 |
esberglu | vSCSI is ready too, but I do want to get a stack with that and test again, make sure it still works | 14:21 |
edmondsw | +1 | 14:21 |
efried | We need to get subteam +1s first; then we can move 'em to broader review and start pestering people. Which means I (at least) need to get on those reviews. | 14:21 |
efried | cause I think I'm behind on at least a couple. | 14:21 |
edmondsw | efried yep, that's why their only marked for subteam review except where you/I have already +1d | 14:21 |
edmondsw | they're* | 14:22 |
edmondsw | good call adding the comments on what each link is | 14:22 |
efried | ++ | 14:23 |
edmondsw | I will try to take another look at the ones ready for subteam review again today | 14:23 |
edmondsw | esberglu update us on the others that are in-progress but not ready yet? | 14:23 |
esberglu | There are 2 more that I have underway, both related to localdisk | 14:24 |
esberglu | https://review.openstack.org/#/c/549053/ | 14:24 |
esberglu | https://review.openstack.org/#/c/549300/ | 14:24 |
esberglu | The first adds the DiskAdapter parent class, currently we just have the ssp adapter in tree | 14:24 |
edmondsw | why isn't 053 ready for subteam review? | 14:24 |
esberglu | It is | 14:25 |
edmondsw | ok, so we can add that to the etherpad | 14:25 |
esberglu | I just didn't add it yet in case snapshot changes will require rebasing / modifying | 14:25 |
esberglu | I'll add it | 14:26 |
edmondsw | looks like it is based on an older snapshot ps | 14:26 |
esberglu | edmondsw: Yeah I was holding off on rebasing until you guys +1 | 14:26 |
edmondsw | works for me | 14:26 |
edmondsw | I think the only other thing I had to mention IT is that esberglu and I cleaned up the TODO etherpad: https://etherpad.openstack.org/p/powervm-in-tree-todos | 14:27 |
edmondsw | I will add a link to the nova rocky etherpad... tx for pointing that out yesterday efried | 14:28 |
edmondsw | anything else for IT? | 14:28 |
esberglu | nope | 14:28 |
edmondsw | #topic Out-of-Tree Driver | 14:29 |
*** openstack changes topic to "Out-of-Tree Driver (Meeting topic: PowerVM Driver Meeting)" | 14:29 | |
edmondsw | new etherpad for this: https://etherpad.openstack.org/p/powervm-oot-todos | 14:29 |
edmondsw | efried I added the volume id item we were discussing yesterday | 14:29 |
edmondsw | SRIOV max capacity merged, so I can remove that | 14:30 |
edmondsw | I spent some more time looking at the volume refactor yesterday | 14:30 |
edmondsw | getting close there... | 14:30 |
edmondsw | I'd like to get reviews on that tied up this week, and see if we can get that tested and merged by end of the month | 14:31 |
edmondsw | efried your volume_id change proposal is pretty small, but we should talk about whether it makes sense to do that on top of the refactor rather than have to do it twice | 14:32 |
edmondsw | thoughts? | 14:32 |
edmondsw | Imran said they can wait a couple weeks on the volume id change | 14:32 |
efried | As you say, it's pretty small, the rebase wouldn't be hard, could do it either side. Since Imran says we want this by end of month, we may want to do the volume ID first. | 14:32 |
efried | oh, if you want to think of it that way :) | 14:33 |
edmondsw | yeah, it is small, so I could go either way | 14:33 |
efried | We need to get the pypowervm side merged and through g-r before it'll pass tests | 14:33 |
edmondsw | was pretty set on it being after the refactor, but then realized how small it is :) | 14:33 |
efried | and core/REST needs more patches before it'll actually work. | 14:33 |
edmondsw | efried yep, I think the g-r change may be the bigger issue | 14:33 |
edmondsw | as far as how quickly we can get this to them | 14:33 |
efried | But those don't really factor into which order we do the n-pvm changes | 14:34 |
edmondsw | and the core/REST changes, of course | 14:34 |
edmondsw | yep | 14:34 |
efried | So - how close are we on The Big Refactor? | 14:34 |
edmondsw | pretty close | 14:34 |
efried | (I of course should review that) | 14:34 |
edmondsw | definitely | 14:34 |
edmondsw | I am doing another pass through the changes, hope to finish that today | 14:35 |
edmondsw | don't expect much more to shake out there | 14:35 |
edmondsw | I did notice something yesterday while digging through... we don't have live migration support for iSCSI | 14:35 |
edmondsw | so I added that to our TODO etherpad after talking to Gerald and confirming that PowerVC will need that | 14:35 |
edmondsw | anything else to talk about OOT? | 14:36 |
esberglu | I have a couple easy reviews out | 14:36 |
esberglu | https://review.openstack.org/#/c/548410/ | 14:37 |
esberglu | https://review.openstack.org/#/c/548412/ | 14:37 |
esberglu | If efried could get a review on those | 14:37 |
esberglu | And give your thoughts on | 14:37 |
esberglu | https://review.openstack.org/#/c/548721/ | 14:37 |
esberglu | that all I had | 14:38 |
edmondsw | looks like someone has raised a docs fix: https://review.openstack.org/#/c/552379/ | 14:39 |
edmondsw | I'll look at that after | 14:39 |
edmondsw | #topic Device Passthrough | 14:40 |
*** openstack changes topic to "Device Passthrough (Meeting topic: PowerVM Driver Meeting)" | 14:40 | |
edmondsw | efried ? | 14:40 |
edmondsw | any updates from the PTG? | 14:40 |
efried | Heaping pile of blueprints/specs/patches from the PTG, plus the update_provider_tree stuff that's been sitting out there for a while. | 14:40 |
edmondsw | I spent some time thinking through use cases and such while you were out... we need to catch up and go over that when you have a few minutes | 14:41 |
efried | Some changes to UPT came out of the PTG; spec delta almost merged (https://review.openstack.org/#/c/552122/) but I need to write the code accordingly. That's "first" on my agenda. | 14:41 |
efried | No news on NRP in GET /a_c; I'll bug jaypipes about that soon. But with those two pieces, we should be pretty free & clear to start proposing our own driver patches to implement UPT. | 14:42 |
edmondsw | I don't think we're going to have that borrowed system for too much longer, so you should probably try to get on that while we can | 14:42 |
efried | Once we have that, we can do generic passthrough. So edmondsw you have time to figure out the ux so we can walk that back to the code. | 14:43 |
edmondsw | figure out what the card looks like as far as identifiers and such | 14:43 |
efried | Oh, was that on me? I thought you were on the hook for that. | 14:43 |
edmondsw | yes and yes | 14:43 |
edmondsw | :) | 14:43 |
edmondsw | I figured out what it should look like | 14:44 |
edmondsw | I haven't gotten on the system to confirm that it actually looks like that... I thought you were doing that | 14:44 |
edmondsw | I can try to figure out how to do that | 14:44 |
efried | Wasn't on my radar. If you have room for it, that would be cool. So the system has GPUs on it? | 14:44 |
edmondsw | yes | 14:45 |
edmondsw | the really nice kind | 14:45 |
edmondsw | Nicolas has it patched so it should work for us now | 14:45 |
efried | So pvmctl sys list --xml maybe | 14:45 |
edmondsw | k | 14:45 |
edmondsw | efried get together this afternoon and go over things? | 14:46 |
efried | But we still need to figure out the set of devices we'll support, and how we're going to represent it, and how we're going to discover. | 14:47 |
edmondsw | that would be in the "go over things" discussion :) | 14:47 |
efried | Yes, let's | 14:47 |
edmondsw | alright, enough on that for now I think | 14:47 |
efried | Send me a calendar notice, and then send me a message on IRC to let me know you've done that. I'm only down to 283 unread messages in Notes. | 14:47 |
edmondsw | efried ack | 14:47 |
efried | (432 and growing in my other client - dev ML) | 14:48 |
edmondsw | #topic Open Discussion | 14:48 |
*** openstack changes topic to "Open Discussion (Meeting topic: PowerVM Driver Meeting)" | 14:48 | |
esberglu | edmondsw: CI? | 14:48 |
edmondsw | oh, oops... | 14:48 |
edmondsw | #topic PowerVM CI | 14:48 |
*** openstack changes topic to "PowerVM CI (Meeting topic: PowerVM Driver Meeting)" | 14:48 | |
edmondsw | esberglu, you're up | 14:48 |
esberglu | CI has been pretty consistent lately | 14:48 |
esberglu | Even with the updated post stack cleaner and https://review.openstack.org/#/c/540453/ | 14:49 |
esberglu | I'm still seeing that vopt issue on occaison on OOT runs | 14:49 |
esberglu | I have some time cut out tomorrow to hop back into that | 14:49 |
efried | Did we try to reinstate the rename patch and see if that kills it? | 14:50 |
esberglu | Yeah the rename patch is being applied to all OOT runs | 14:50 |
edmondsw | I think that's the link above | 14:50 |
efried | Okay, so even with that, we're still hitting it? | 14:50 |
esberglu | yep | 14:50 |
efried | oh, sorry, yeah. | 14:50 |
efried | So hm, curiouser and curiouser. Good luck with that. | 14:50 |
edmondsw | interesting that we're still seeing it with that | 14:51 |
esberglu | Still working the CI undercloud upgrade, decided to jump from ocata to queens instead of ocata to pike | 14:51 |
esberglu | I have a working queens cloud, but the management services aren't playing nice with it | 14:52 |
esberglu | Because we are on old versions of those that are having some issues with the identity v3 api | 14:52 |
edmondsw | what? | 14:52 |
esberglu | The version of nodepool we are on doesn't work with the new identity api | 14:53 |
esberglu | And needs to be upgraded as well | 14:53 |
edmondsw | that gonna be a problem? | 14:53 |
esberglu | edmondsw: It will also mean updating zuul and jenkins and adding a zookeeper service and maybe some other stuff | 14:53 |
edmondsw | maybe that's a good thing... it'll be nice to be more current on everything | 14:53 |
edmondsw | more work though | 14:54 |
esberglu | edmondsw: I'm trying a queens deploy using identity v2 api for now | 14:54 |
esberglu | Just to verify that queens works otherwise | 14:54 |
edmondsw | huh... I thought v2 was removed in queens | 14:54 |
esberglu | Oh maybe, I thought I saw it in there, could have been looking at the wrong thing | 14:55 |
edmondsw | I wasn't very involved with keystone during queens, though, so I could have gotten that wrong | 14:56 |
esberglu | But yeah, still just trying to get something working there, I'll keep you posted on progress | 14:56 |
edmondsw | tx | 14:56 |
edmondsw | anything else? | 14:56 |
esberglu | I'm starting to look forward into the multinode CI / OVS CI | 14:56 |
edmondsw | +1 | 14:57 |
esberglu | Nothing to talk about there yet though | 14:57 |
edmondsw | #topic Open Discussion | 14:58 |
*** openstack changes topic to "Open Discussion (Meeting topic: PowerVM Driver Meeting)" | 14:58 | |
edmondsw | I wanted to note a couple things with the PowerStackers group | 14:58 |
edmondsw | tonyb has proposed renaming to PowerVMStackers, making it PowerVM specific again like my original proposal | 14:58 |
edmondsw | https://review.openstack.org/#/c/551413 | 14:59 |
edmondsw | I replied to that this morning | 14:59 |
edmondsw | I suspect it will get renamed, and I'm not sure I disagree with that | 14:59 |
efried | I'm writing up a concurring opinion atm | 15:00 |
edmondsw | he also raised an objection on the meeting proposal: https://review.openstack.org/#/c/552132 | 15:00 |
edmondsw | I replied to that as well seeking some clarification | 15:00 |
edmondsw | it sounded like he might be saying we should just meet in our channel because the official meeting channels are being deemphasized? but I'm not sure I got that right | 15:01 |
edmondsw | so we'll see where that goes | 15:01 |
edmondsw | anything else to discuss? | 15:01 |
esberglu | nope | 15:02 |
edmondsw | #endmeeting | 15:02 |
*** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 15:02 | |
openstack | Meeting ended Tue Mar 13 15:02:46 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:02 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-03-13-14.00.html | 15:02 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-03-13-14.00.txt | 15:02 |
openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-03-13-14.00.log.html | 15:02 |
efried | edmondsw: Posted (https://review.openstack.org/#/c/551413/) | 15:05 |
edmondsw | efried yep, tx | 15:07 |
edmondsw | I went ahead and bumped myself to +1 as well | 15:08 |
*** tjakobs has joined #openstack-powervm | 15:20 | |
edmondsw | esberglu I didn't see multinode on the CI todo etherpad, so I added it | 15:22 |
esberglu | tx | 15:22 |
edmondsw | esberglu want to fix that typo on https://review.openstack.org/#/c/548721 or should I just merge it? | 15:48 |
esberglu | edmondsw: I can fix it quick | 15:49 |
*** tblakes has quit IRC | 15:50 | |
efried | edmondsw: Feel free to fast-approve. No need to wait for me to re+2 | 15:56 |
edmondsw | efried yep, I was planning to | 15:56 |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Mock retry in test_remove_block_dev https://review.openstack.org/548721 | 16:01 |
openstackgerrit | Merged openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/552309 | 16:09 |
openstackgerrit | Merged openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/552364 | 16:22 |
openstackgerrit | Merged openstack/nova-powervm master: Update taskflow instantiation https://review.openstack.org/548410 | 16:22 |
openstackgerrit | Merged openstack/nova-powervm master: Add missing tasks tests https://review.openstack.org/548412 | 16:22 |
openstackgerrit | Merged openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/552355 | 16:22 |
*** AlexeyAbashkin has quit IRC | 17:34 | |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: ci check https://review.openstack.org/328315 | 17:57 |
openstackgerrit | Merged openstack/nova-powervm master: Mock retry in test_remove_block_dev https://review.openstack.org/548721 | 18:20 |
*** AlexeyAbashkin has joined #openstack-powervm | 18:58 | |
*** AlexeyAbashkin has quit IRC | 19:03 | |
*** esberglu has quit IRC | 19:27 | |
*** esberglu has joined #openstack-powervm | 19:37 | |
openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Set volume ID as PV tag when mapping VSCSI volumes https://review.openstack.org/552172 | 20:18 |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 20:18 |
*** esberglu has quit IRC | 21:30 | |
*** esberglu has joined #openstack-powervm | 21:33 | |
*** AlexeyAbashkin has joined #openstack-powervm | 21:36 | |
*** esberglu has quit IRC | 21:37 | |
*** AlexeyAbashkin has quit IRC | 21:41 | |
*** esberglu has joined #openstack-powervm | 22:02 | |
tonyb | edmondsw: Yeah I didn't realise that old requirement still existised after the TC voted to allow meetings in project channels. So yeah you can hold your meeting here or in a -meeting-{3,4} either is fine | 22:10 |
tonyb | the latter is good if you want to highlight/ping people but that works best when there is TZ overlap | 22:10 |
edmondsw | tonyb sure... efried esberglu y'all have an opinion? | 22:11 |
edmondsw | I'll probably just abandon that and keep having it here for now unless someone speaks up | 22:11 |
efried | I thought there was some benefit to having it in a "real" channel. | 22:12 |
esberglu | I'd prefer here but don't feel strongly either way | 22:12 |
tonyb | edmondsw: Rather than abanond it just chnage the channel to here | 22:12 |
tonyb | that way you get listed on the ical/eavesdrop page | 22:12 |
edmondsw | tonyb ah, good call | 22:12 |
efried | That ^ | 22:12 |
edmondsw | will do | 22:12 |
tonyb | efried: There certainly used to be and I disagree with the TC's move to encourage meetings in per project channels but at the same time I understand the motivation. It's a shame they missed that old blurb | 22:13 |
tonyb | efried: I'll fix that today if it hasn't already been done by ttx ;P | 22:14 |
tonyb | edmondsw, efried I can do it for you and reference this log and then +W the change | 22:16 |
edmondsw | tonyb can do what? | 22:17 |
edmondsw | I just reproposed pointing to this channel as the meeting place | 22:17 |
tonyb | edmondsw: propose an alteration to you irc-meetings change setting the channel to here | 22:17 |
tonyb | edmondsw: Oh okay packest passing in the ether | 22:17 |
edmondsw | :) | 22:17 |
tonyb | edmondsw: as long as tox likes it it'll merge in ~10mins | 22:18 |
edmondsw | tonyb great, tx | 22:18 |
tonyb | np | 22:18 |
*** edmondsw has quit IRC | 22:21 | |
*** apearson_ has quit IRC | 22:32 | |
*** tjakobs has quit IRC | 22:45 | |
*** AlexeyAbashkin has joined #openstack-powervm | 23:36 | |
*** edmondsw has joined #openstack-powervm | 23:36 | |
*** AlexeyAbashkin has quit IRC | 23:41 | |
*** edmondsw has quit IRC | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!