openstackgerrit | Merged openstack/nova-powervm master: Mock sleep in UT https://review.openstack.org/539023 | 00:32 |
---|---|---|
*** svenkat has joined #openstack-powervm | 00:51 | |
*** svenkat has quit IRC | 00:55 | |
*** svenkat has joined #openstack-powervm | 01:11 | |
*** efried_hexchat has quit IRC | 02:06 | |
*** svenkat has quit IRC | 02:07 | |
*** efried_hexchat has joined #openstack-powervm | 02:19 | |
*** edmondsw has joined #openstack-powervm | 02:54 | |
*** edmondsw has quit IRC | 02:58 | |
*** edmondsw has joined #openstack-powervm | 04:42 | |
*** edmondsw has quit IRC | 04:46 | |
*** esberglu_ has joined #openstack-powervm | 05:51 | |
*** esberglu has quit IRC | 05:55 | |
*** edmondsw has joined #openstack-powervm | 06:30 | |
*** AlexeyAbashkin has joined #openstack-powervm | 06:33 | |
*** edmondsw has quit IRC | 06:34 | |
*** AlexeyAbashkin has quit IRC | 06:37 | |
*** esberglu has joined #openstack-powervm | 06:55 | |
*** esberglu_ has quit IRC | 06:57 | |
*** AlexeyAbashkin has joined #openstack-powervm | 07:41 | |
*** AlexeyAbashkin has quit IRC | 07:46 | |
*** AlexeyAbashkin has joined #openstack-powervm | 07:50 | |
*** edmondsw has joined #openstack-powervm | 08:18 | |
*** edmondsw has quit IRC | 08:23 | |
*** AlexeyAbashkin has quit IRC | 08:35 | |
*** AlexeyAbashkin has joined #openstack-powervm | 08:38 | |
*** AlexeyAbashkin has quit IRC | 08:41 | |
*** AlexeyAbashkin has joined #openstack-powervm | 08:42 | |
*** edmondsw has joined #openstack-powervm | 10:06 | |
*** edmondsw has quit IRC | 10:10 | |
*** edmondsw has joined #openstack-powervm | 10:12 | |
*** edmondsw has quit IRC | 10:12 | |
*** AlexeyAbashkin has quit IRC | 10:50 | |
*** AlexeyAbashkin has joined #openstack-powervm | 10:51 | |
*** AlexeyAbashkin has quit IRC | 10:56 | |
*** AlexeyAbashkin has joined #openstack-powervm | 11:22 | |
*** AlexeyAbashkin has quit IRC | 11:51 | |
*** AlexeyAbashkin has joined #openstack-powervm | 11:52 | |
*** edmondsw has joined #openstack-powervm | 12:08 | |
*** edmondsw_ has joined #openstack-powervm | 12:49 | |
*** svenkat has joined #openstack-powervm | 12:49 | |
*** edmondsw has quit IRC | 12:51 | |
*** edmondsw_ has quit IRC | 12:58 | |
*** edmondsw has joined #openstack-powervm | 13:00 | |
-openstackstatus- NOTICE: Our ubuntu-xenial images (used for e.g. unit tests and devstack) are currently failing to install any packages, restrain from *recheck* or *approve* until the issue has been investigated and fixed. | 13:43 | |
*** apearson has joined #openstack-powervm | 13:59 | |
esberglu | #startmeeting powervm_driver_meeting | 14:00 |
openstack | Meeting started Tue Jan 30 14:00:01 2018 UTC and is due to finish in 60 minutes. The chair is esberglu. 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 |
efried | @/ | 14:00 |
edmondsw | o/ | 14:01 |
esberglu | #topic In-tree Driver | 14:01 |
*** openstack changes topic to "In-tree Driver (Meeting topic: powervm_driver_meeting)" | 14:01 | |
esberglu | Nothing here from me | 14:02 |
edmondsw | so we got config drive, OVS, and SEA merged for Queens | 14:02 |
esberglu | Unless we want to start looking at plans moving into Rocky | 14:02 |
edmondsw | but not vSCSI | 14:02 |
esberglu | Yep | 14:02 |
efried | We can propose the Rocky bp any time now. | 14:02 |
edmondsw | I think our goals for Rocky will be... | 14:02 |
edmondsw | a) vSCSI | 14:02 |
edmondsw | b) migrate | 14:02 |
edmondsw | c) resize | 14:02 |
edmondsw | d) snapshot | 14:02 |
edmondsw | any comments on that? | 14:03 |
efried | c)? Did we decide we can't already do that? | 14:03 |
efried | oh, sorry, resize meaning migrate-to-same-host :( | 14:03 |
edmondsw | as in, change the flavor of an instance | 14:03 |
edmondsw | yes, would stay on the same host | 14:03 |
esberglu | Sounds good to me | 14:04 |
efried | Do we have localdisk driver in tree? | 14:04 |
efried | And... do we care about that? | 14:04 |
esberglu | efried: We do not | 14:04 |
edmondsw | I'm also looking at volume multiattach... it may be almost a no-op that we could squeeze in with the vSCSI support | 14:04 |
esberglu | There is a WIP patch for localdisk https://review.openstack.org/#/c/409401/ | 14:05 |
edmondsw | efried you mean local volumes or local boot disk? | 14:05 |
efried | boot | 14:05 |
edmondsw | I thought that was already inm | 14:05 |
edmondsw | hmm... will need to take a look at that and see how much more there is to do there | 14:06 |
edmondsw | maybe we can squeeze that in as well | 14:06 |
efried | The other thing we ought to have on the slate is porting from get_available_resource to update_provider_tree during the Rocky time frame. Both in-tree and out. The framework stuff (that we didn't quite finish in Queens) ought to go in relatively early, and I want us to be right on top of it. | 14:06 |
efried | I would prioritize that over some of those other things. | 14:07 |
edmondsw | efried sure, and I'm hoping you can run with that | 14:07 |
edmondsw | I'm thinking that's a separate effort from the port-in bp | 14:08 |
edmondsw | i.e. if needs a bp, would be a separate bp | 14:08 |
edmondsw | thoughts? | 14:08 |
efried | Agree it's a separate effort, yes. And yes, probably bp-worthy. | 14:09 |
esberglu | edmondsw: It makes sense to be a separate bp | 14:09 |
edmondsw | so we're all on the same page there | 14:09 |
esberglu | I'll add some notes to the etherpad with our current rocky plans | 14:10 |
efried | cool | 14:10 |
edmondsw | sounds good | 14:10 |
esberglu | Anything else to discuss IT? | 14:10 |
edmondsw | let's hold off on writing the bp for a few days | 14:10 |
edmondsw | that's it | 14:11 |
esberglu | #topic Out-of-tree Driver | 14:11 |
*** openstack changes topic to "Out-of-tree Driver (Meeting topic: powervm_driver_meeting)" | 14:11 | |
esberglu | The queens-3 tags are now released for all 3 projects so they | 14:11 |
esberglu | are also in feature freeze now | 14:11 |
esberglu | There's a new changeset up for the volume refactor, haven't had a chance to look at that yet | 14:12 |
edmondsw | esberglu yep, I saw that. hoping to look at it today | 14:13 |
edmondsw | but we couldn't merge until Rocky? | 14:13 |
esberglu | edmondsw: Technically we run the project so we have the power to push stuff during ff | 14:13 |
esberglu | But it's frowned upon | 14:14 |
efried | Not "couldn't", but shouldn't, yeah. | 14:14 |
edmondsw | yeah, I feel better holding off on that to Rocky | 14:14 |
edmondsw | not that we can't go ahead and review | 14:14 |
edmondsw | I kinda expect another iteration or two | 14:14 |
esberglu | yep | 14:15 |
esberglu | I think that's about it going on OOT atm | 14:16 |
edmondsw | yep | 14:16 |
esberglu | #topic Device Passthrough | 14:17 |
*** openstack changes topic to "Device Passthrough (Meeting topic: powervm_driver_meeting)" | 14:17 | |
efried | We're going to have a meeting later. | 14:18 |
efried | We got a substantial amount done in Q, but the bits that tie it all together are still pending. | 14:18 |
edmondsw | I hope to come out of that mtg with a much better understanding of what we need to do for the bp mentioned above | 14:18 |
efried | One such piece is NRP in GET /allocation_candidates. Which is a big deal for NRP - but not really such a big deal for Power. I'm not sure we're going to need to model with nested, really. | 14:18 |
efried | Which would be good, because that's the part that's actually the furthest behind. (Jay owns it) | 14:19 |
efried | The part that I think we can really use is the update_provider_tree business. | 14:19 |
efried | There are six approved patches thrashing in the gate still. But the cherry on top is the next few patches, which I just finished up yesterday, and should go in early in Q | 14:20 |
efried | but also | 14:20 |
efried | we could easily carry those in pvcos if we wanted to make use of it right away. | 14:20 |
edmondsw | s/Q/R/ | 14:20 |
efried | yeah | 14:20 |
edmondsw | glad to hear you say that. I was hoping we could carry things in pvcos without too much trouble if need be | 14:21 |
edmondsw | we'll talk more in a few hours | 14:21 |
efried | It's relatively little code, couple hundred lines. And it works. | 14:21 |
edmondsw | working is good... | 14:22 |
efried | yeah | 14:22 |
efried | If you want to see it working, check out this functional test suite: https://review.openstack.org/#/c/520246/42/nova/tests/functional/test_servers.py@1594 | 14:23 |
edmondsw | next topic? | 14:24 |
esberglu | #topic PowerVM CI | 14:25 |
*** openstack changes topic to "PowerVM CI (Meeting topic: powervm_driver_meeting)" | 14:25 | |
esberglu | I've been upgrading staging to pike | 14:25 |
esberglu | I have a mostly working stack | 14:25 |
esberglu | But can't get snapshots to work yet | 14:25 |
esberglu | They just get stuck in queued status and never move anywhere | 14:26 |
*** AlexeyAbashkin has quit IRC | 14:26 | |
*** AlexeyAbashkin has joined #openstack-powervm | 14:27 | |
esberglu | The management services work fine with the new undercloud afaik, but once the image template builds there's no snapshot so it can't move past that | 14:27 |
esberglu | Hoping that once I get snapshot working the rest will be fine | 14:27 |
edmondsw | good luck | 14:27 |
esberglu | Other than that I still haven't figured out what's going on with that update server name non-ascii test | 14:28 |
esberglu | Haven't been able to reproduce it on a test instance | 14:28 |
esberglu | The last item is the multiple unplug calls on instance deletion | 14:30 |
esberglu | I might bug thorst about that, I think he probably knows more about _push_vif_event | 14:31 |
edmondsw | sure. Not a huge priority, but would be nice to clean up a bit | 14:31 |
esberglu | Yep. And there's one other error I've been seeing in a few runs trying to delete vopts | 14:32 |
esberglu | "Unable to perform operation while the virtual DVD is loaded" | 14:32 |
esberglu | Haven't started debugging that one yet | 14:32 |
edmondsw | k | 14:32 |
esberglu | #topic Open Discussion | 14:32 |
*** openstack changes topic to "Open Discussion (Meeting topic: powervm_driver_meeting)" | 14:33 | |
esberglu | Nothing else from me | 14:33 |
edmondsw | nor here | 14:33 |
efried | I can't think of anything | 14:33 |
esberglu | #endmeeting | 14:34 |
*** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 14:34 | |
openstack | Meeting ended Tue Jan 30 14:34:36 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:34 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-30-14.00.html | 14:34 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-30-14.00.txt | 14:34 |
openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-30-14.00.log.html | 14:34 |
*** svenkat has quit IRC | 14:57 | |
efried | edmondsw: Re https://review.openstack.org/#/c/538330/ -W will get cleared if you propose a new patch set. -2 will stick. | 15:14 |
efried | So this way we can review and keep proposing new patches until we're ready. | 15:15 |
edmondsw | ah, ok | 15:15 |
edmondsw | removing the -W then | 15:16 |
*** tjakobs has joined #openstack-powervm | 15:17 | |
esberglu | edmondsw: efried: See 6275 when you get a chance. I'm taking a CI system for development | 15:24 |
edmondsw | ack | 15:24 |
*** tjakobs has quit IRC | 15:39 | |
*** AlexeyAbashkin has quit IRC | 15:50 | |
*** AlexeyAbashkin has joined #openstack-powervm | 15:50 | |
*** AlexeyAbashkin has quit IRC | 15:55 | |
*** AlexeyAbashkin has joined #openstack-powervm | 15:55 | |
*** svenkat has joined #openstack-powervm | 16:21 | |
*** svenkat_ has joined #openstack-powervm | 16:25 | |
*** svenkat has quit IRC | 16:25 | |
*** svenkat_ is now known as svenkat | 16:25 | |
*** AlexeyAbashkin has quit IRC | 16:38 | |
*** AlexeyAbashkin has joined #openstack-powervm | 17:29 | |
*** AlexeyAbashkin has quit IRC | 18:46 | |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf https://review.openstack.org/539311 | 20:31 |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf https://review.openstack.org/539311 | 20:32 |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf https://review.openstack.org/539311 | 21:01 |
*** apearson has quit IRC | 21:48 | |
*** svenkat has quit IRC | 22:01 | |
*** edmondsw has quit IRC | 22:11 | |
*** edmondsw has joined #openstack-powervm | 22:12 | |
openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Add supports_multiattach to capabilities https://review.openstack.org/539344 | 22:14 |
esberglu | efried: edmondsw: ^ We need that to avoid getting KeyErrors | 22:14 |
*** edmondsw has quit IRC | 22:17 | |
esberglu | efried: Why do we not have capabilities for the IT driver? Am I missing something? | 22:18 |
efried | The capabilities dict at the top of the compute driver subclass? | 22:19 |
efried | Presumably because all the defaults happen to be true. | 22:19 |
efried | Soon as we add a capability that's False in the superclass dict, we should override. | 22:20 |
efried | Though arguably we should override anyway. | 22:20 |
esberglu | efried: So we shouldn't actually need that OOT patch. I definitely got a KeyError for supports_multiattch though | 22:23 |
efried | Actually, the capabilities dict is goofy. Because you can't partially override it. You have to go all or nothing. | 22:24 |
efried | If they've added a new key to the base class since the last time we updated, and they're not doing a safe get of the value, then yeah, we have to add it to ours. | 22:25 |
efried | Though I would prefer proposing a nova patch to make them do a safe get. That's the kind of irresponsible shit we as third-party driver vendors shouldn't have to dork with. | 22:25 |
efried | Sure enough, ildikov added it to all the in-tree drivers. | 22:26 |
efried | esberglu: Fix is in nova/virt/block_device.py around line 500. Change that to .get('supports_multiattach', False) and we're good. | 22:28 |
esberglu | efried: That the only place? I was just pulling latest nova to check | 22:28 |
efried | Open a bug and submit a patch. | 22:28 |
efried | Um, I think so. Let me look closer... | 22:28 |
efried | No, compute manager too. And a passel of test cases. | 22:29 |
esberglu | efried: Okay. I've got to head out for the day but will take care of it tomorrow | 22:30 |
efried | esberglu: Open a bug, though. That'll be good impetus to get it in. | 22:30 |
*** esberglu has quit IRC | 22:32 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!