Tuesday, 2018-01-30

openstackgerritMerged openstack/nova-powervm master: Mock sleep in UT  https://review.openstack.org/53902300:32
*** svenkat has joined #openstack-powervm00:51
*** svenkat has quit IRC00:55
*** svenkat has joined #openstack-powervm01:11
*** efried_hexchat has quit IRC02:06
*** svenkat has quit IRC02:07
*** efried_hexchat has joined #openstack-powervm02:19
*** edmondsw has joined #openstack-powervm02:54
*** edmondsw has quit IRC02:58
*** edmondsw has joined #openstack-powervm04:42
*** edmondsw has quit IRC04:46
*** esberglu_ has joined #openstack-powervm05:51
*** esberglu has quit IRC05:55
*** edmondsw has joined #openstack-powervm06:30
*** AlexeyAbashkin has joined #openstack-powervm06:33
*** edmondsw has quit IRC06:34
*** AlexeyAbashkin has quit IRC06:37
*** esberglu has joined #openstack-powervm06:55
*** esberglu_ has quit IRC06:57
*** AlexeyAbashkin has joined #openstack-powervm07:41
*** AlexeyAbashkin has quit IRC07:46
*** AlexeyAbashkin has joined #openstack-powervm07:50
*** edmondsw has joined #openstack-powervm08:18
*** edmondsw has quit IRC08:23
*** AlexeyAbashkin has quit IRC08:35
*** AlexeyAbashkin has joined #openstack-powervm08:38
*** AlexeyAbashkin has quit IRC08:41
*** AlexeyAbashkin has joined #openstack-powervm08:42
*** edmondsw has joined #openstack-powervm10:06
*** edmondsw has quit IRC10:10
*** edmondsw has joined #openstack-powervm10:12
*** edmondsw has quit IRC10:12
*** AlexeyAbashkin has quit IRC10:50
*** AlexeyAbashkin has joined #openstack-powervm10:51
*** AlexeyAbashkin has quit IRC10:56
*** AlexeyAbashkin has joined #openstack-powervm11:22
*** AlexeyAbashkin has quit IRC11:51
*** AlexeyAbashkin has joined #openstack-powervm11:52
*** edmondsw has joined #openstack-powervm12:08
*** edmondsw_ has joined #openstack-powervm12:49
*** svenkat has joined #openstack-powervm12:49
*** edmondsw has quit IRC12:51
*** edmondsw_ has quit IRC12:58
*** edmondsw has joined #openstack-powervm13: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-powervm13:59
esberglu#startmeeting powervm_driver_meeting14:00
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: powervm_driver_meeting)"14:00
openstackThe meeting name has been set to 'powervm_driver_meeting'14:00
esberglu#topic In-tree Driver14:01
*** openstack changes topic to "In-tree Driver (Meeting topic: powervm_driver_meeting)"14:01
esbergluNothing here from me14:02
edmondswso we got config drive, OVS, and SEA merged for Queens14:02
esbergluUnless we want to start looking at plans moving into Rocky14:02
edmondswbut not vSCSI14:02
efriedWe can propose the Rocky bp any time now.14:02
edmondswI think our goals for Rocky will be...14:02
edmondswa) vSCSI14:02
edmondswb) migrate14:02
edmondswc) resize14:02
edmondswd) snapshot14:02
edmondswany comments on that?14:03
efriedc)? Did we decide we can't already do that?14:03
efriedoh, sorry, resize meaning migrate-to-same-host :(14:03
edmondswas in, change the flavor of an instance14:03
edmondswyes, would stay on the same host14:03
esbergluSounds good to me14:04
efriedDo we have localdisk driver in tree?14:04
efriedAnd... do we care about that?14:04
esbergluefried: We do not14:04
edmondswI'm also looking at volume multiattach... it may be almost a no-op that we could squeeze in with the vSCSI support14:04
esbergluThere is a WIP patch for localdisk https://review.openstack.org/#/c/409401/14:05
edmondswefried you mean local volumes or local boot disk?14:05
edmondswI thought that was already inm14:05
edmondswhmm... will need to take a look at that and see how much more there is to do there14:06
edmondswmaybe we can squeeze that in as well14:06
efriedThe 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
efriedI would prioritize that over some of those other things.14:07
edmondswefried sure, and I'm hoping you can run with that14:07
edmondswI'm thinking that's a separate effort from the port-in bp14:08
edmondswi.e. if needs a bp, would be a separate bp14:08
efriedAgree it's a separate effort, yes.  And yes, probably bp-worthy.14:09
esbergluedmondsw: It makes sense to be a separate bp14:09
edmondswso we're all on the same page there14:09
esbergluI'll add some notes to the etherpad with our current rocky plans14:10
edmondswsounds good14:10
esbergluAnything else to discuss IT?14:10
edmondswlet's hold off on writing the bp for a few days14:10
edmondswthat's it14:11
esberglu#topic Out-of-tree Driver14:11
*** openstack changes topic to "Out-of-tree Driver (Meeting topic: powervm_driver_meeting)"14:11
esbergluThe queens-3 tags are now released for all 3 projects so they14:11
esbergluare also in feature freeze now14:11
esbergluThere's a new changeset up for the volume refactor, haven't had a chance to look at that yet14:12
edmondswesberglu yep, I saw that. hoping to look at it today14:13
edmondswbut we couldn't merge until Rocky?14:13
esbergluedmondsw: Technically we run the project so we have the power to push stuff during ff14:13
esbergluBut it's frowned upon14:14
efriedNot "couldn't", but shouldn't, yeah.14:14
edmondswyeah, I feel better holding off on that to Rocky14:14
edmondswnot that we can't go ahead and review14:14
edmondswI kinda expect another iteration or two14:14
esbergluI think that's about it going on OOT atm14:16
esberglu#topic Device Passthrough14:17
*** openstack changes topic to "Device Passthrough (Meeting topic: powervm_driver_meeting)"14:17
efriedWe're going to have a meeting later.14:18
efriedWe got a substantial amount done in Q, but the bits that tie it all together are still pending.14:18
edmondswI hope to come out of that mtg with a much better understanding of what we need to do for the bp mentioned above14:18
efriedOne 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
efriedWhich would be good, because that's the part that's actually the furthest behind.  (Jay owns it)14:19
efriedThe part that I think we can really use is the update_provider_tree business.14:19
efriedThere 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 Q14:20
efriedbut also14:20
efriedwe could easily carry those in pvcos if we wanted to make use of it right away.14:20
edmondsw s/Q/R/14:20
edmondswglad to hear you say that. I was hoping we could carry things in pvcos without too much trouble if need be14:21
edmondswwe'll talk more in a few hours14:21
efriedIt's relatively little code, couple hundred lines.  And it works.14:21
edmondswworking is good...14:22
efriedIf 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@159414:23
edmondswnext topic?14:24
esberglu#topic PowerVM CI14:25
*** openstack changes topic to "PowerVM CI (Meeting topic: powervm_driver_meeting)"14:25
esbergluI've been upgrading staging to pike14:25
esbergluI have a mostly working stack14:25
esbergluBut can't get snapshots to work yet14:25
esbergluThey just get stuck in queued status and never move anywhere14:26
*** AlexeyAbashkin has quit IRC14:26
*** AlexeyAbashkin has joined #openstack-powervm14:27
esbergluThe 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 that14:27
esbergluHoping that once I get snapshot working the rest will be fine14:27
edmondswgood luck14:27
esbergluOther than that I still haven't figured out what's going on with that update server name non-ascii test14:28
esbergluHaven't been able to reproduce it on a test instance14:28
esbergluThe last item is the multiple unplug calls on instance deletion14:30
esbergluI might bug thorst about that, I think he probably knows more about _push_vif_event14:31
edmondswsure. Not a huge priority, but would be nice to clean up a bit14:31
esbergluYep. And there's one other error I've been seeing in a few runs trying to delete vopts14:32
esberglu"Unable to perform operation while the virtual DVD is loaded"14:32
esbergluHaven't started debugging that one yet14:32
esberglu#topic Open Discussion14:32
*** openstack changes topic to "Open Discussion (Meeting topic: powervm_driver_meeting)"14:33
esbergluNothing else from me14:33
edmondswnor here14:33
efriedI can't think of anything14:33
*** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack."14:34
openstackMeeting ended Tue Jan 30 14:34:36 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:34
openstackMinutes:        http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-30-14.00.html14:34
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-30-14.00.txt14:34
openstackLog:            http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-01-30-14.00.log.html14:34
*** svenkat has quit IRC14:57
efriededmondsw: Re https://review.openstack.org/#/c/538330/  -W will get cleared if you propose a new patch set.  -2 will stick.15:14
efriedSo this way we can review and keep proposing new patches until we're ready.15:15
edmondswah, ok15:15
edmondswremoving the -W then15:16
*** tjakobs has joined #openstack-powervm15:17
esbergluedmondsw: efried: See 6275 when you get a chance. I'm taking a CI system for development15:24
*** tjakobs has quit IRC15:39
*** AlexeyAbashkin has quit IRC15:50
*** AlexeyAbashkin has joined #openstack-powervm15:50
*** AlexeyAbashkin has quit IRC15:55
*** AlexeyAbashkin has joined #openstack-powervm15:55
*** svenkat has joined #openstack-powervm16:21
*** svenkat_ has joined #openstack-powervm16:25
*** svenkat has quit IRC16:25
*** svenkat_ is now known as svenkat16:25
*** AlexeyAbashkin has quit IRC16:38
*** AlexeyAbashkin has joined #openstack-powervm17:29
*** AlexeyAbashkin has quit IRC18:46
openstackgerritEric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf  https://review.openstack.org/53931120:31
openstackgerritEric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf  https://review.openstack.org/53931120:32
openstackgerritEric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf  https://review.openstack.org/53931121:01
*** apearson has quit IRC21:48
*** svenkat has quit IRC22:01
*** edmondsw has quit IRC22:11
*** edmondsw has joined #openstack-powervm22:12
openstackgerritEric Berglund proposed openstack/nova-powervm master: Add supports_multiattach to capabilities  https://review.openstack.org/53934422:14
esbergluefried: edmondsw: ^ We need that to avoid getting KeyErrors22:14
*** edmondsw has quit IRC22:17
esbergluefried: Why do we not have capabilities for the IT driver? Am I missing something?22:18
efriedThe capabilities dict at the top of the compute driver subclass?22:19
efriedPresumably because all the defaults happen to be true.22:19
efriedSoon as we add a capability that's False in the superclass dict, we should override.22:20
efriedThough arguably we should override anyway.22:20
esbergluefried: So we shouldn't actually need that OOT patch. I definitely got a KeyError for supports_multiattch though22:23
efriedActually, the capabilities dict is goofy.  Because you can't partially override it.  You have to go all or nothing.22:24
efriedIf 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
efriedThough 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
efriedSure enough, ildikov added it to all the in-tree drivers.22:26
efriedesberglu: Fix is in nova/virt/block_device.py around line 500.  Change that to .get('supports_multiattach', False) and we're good.22:28
esbergluefried: That the only place? I was just pulling latest nova to check22:28
efriedOpen a bug and submit a patch.22:28
efriedUm, I think so.  Let me look closer...22:28
efriedNo, compute manager too.  And a passel of test cases.22:29
esbergluefried: Okay. I've got to head out for the day but will take care of it tomorrow22:30
efriedesberglu: Open a bug, though.  That'll be good impetus to get it in.22:30
*** esberglu has quit IRC22:32

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!