Wednesday, 2018-08-15

*** openstack has joined #openstack-powervm02:35
*** ChanServ sets mode: +o openstack02:35
*** esberglu has quit IRC13:44
*** esberglu has joined #openstack-powervm13:45
*** esberglu has quit IRC13:50
*** esberglu has joined #openstack-powervm13:56
openstackgerritMatthew Edmonds proposed openstack/ceilometer-powervm master: docs: Modernize build process
efriededmondsw: Does pvc allow hot-plugging SR-IOV VFs to active VMs?14:13
edmondswI believe so, but I'll confirm14:14
edmondswefried yes, confirmed14:15
efriededmondsw: Do we do it via the attach_interface API? If so, then we'll need to hold a local revert of when it lands :(14:16
efriedand/or possibly address the allocation problem some other way?14:16
edmondswefried yes it would be using attach_interface14:18
edmondswefried I'm trying to determine if this will also break nova-powervm/networking-powervm. I suspect yes14:27
edmondswif so, I think we can make a good case that that commit is not hypervisor agnostic and should not merge14:27
efriedIf there's an alloc accounting snafu, we may get steamrolled14:28
efriedbut yeah, would be worth a shot.14:28
efriedcause off the cuff, I don't think they're denying it takes away capability.14:29
edmondswI think we're reading it differently14:29
edmondswI thought mriedem was implying it would fail anyway, and he just wanted it to fail fast14:30
edmondswlet me go look at the bug...14:30
edmondswefried talking to erlarese and yes, it sounds like it would break nova-powervm/networking-powervm's SRIOV attach capability14:33
efriedThe push back is going to look like one or more of:14:34
efried- Your support is OOT and we don't care about OOT14:34
efried- The spec says we don't support it, so we shouldn't support it14:34
edmondswwe support SEA and OVS with the IT driver... is there more work to support SRIOV there?14:34
efriedI think so14:35
edmondswI'm guessing yes... ok14:35
efriedWe have a PvmVnicSriovVifDriver in nova-powervm, so yeah.14:36
efriedWe may be able to argue back that14:36
efried- specs aren't support statements14:36
efried- you can't take away working function without a microversion14:36
efriedAnd maybe suggest that we should instead make a support matrix block for sriov hot-plug and put 'missing' everywhere14:37
efriedinstead of doing this fast-fail14:37
efriedbecause then we (nova-powervm) could claim support (at least partial) in our support matrix.14:37
efriedDo you want to do this -1?14:37
edmondswyes, I'm working on it14:37
edmondswtx for raising it14:37
efriedI think it would play best from you as the nova-powervm ptl. And I can jump in and support you as needed.14:38
edmondswefried comment is up14:43
edmondswtried to keep it short and sweet14:43
efriednice, I like it.14:44
*** gman-tx has joined #openstack-powervm16:16
gman-txefried are you ready for me to +2 this?
efriedgman-tx: You could try. I would be happy with a +1 though :)  (It's nova)16:20
openstackgerritEric Fried proposed openstack/nova-powervm master: PowerVM: update_provider_tree() (compatible)
gman-txi can only +1 in nova ...16:22
efriedgman-tx: You could +2 this one (<== edmondsw fixed the import order)16:22
gman-txefried will have a look16:24
efriedgman-tx: Thx. It's the same patch, just in nova-powervm.16:24
gman-txefried deja vu16:26
efriedthx gman-tx16:28
*** efried is now known as efried_rollin16:28
gman-txwelcome efried_rollin16:29
*** openstackstatus has joined #openstack-powervm16:57
*** ChanServ sets mode: +v openstackstatus16:57
openstackgerritMerged openstack/nova-powervm master: PowerVM: update_provider_tree() (compatible)
*** gman-tx has quit IRC17:31
*** openstackgerrit has quit IRC19:19
*** efried_rollin is now known as efried19:53
*** openstackgerrit has joined #openstack-powervm20:07
openstackgerritEric Fried proposed openstack/nova-powervm master: Inventory whitelist schema and loading
openstackgerritEric Fried proposed openstack/nova-powervm master: Filter IOSlots per inventory config yaml
openstackgerritEric Fried proposed openstack/nova-powervm master: Placement-isms for inventory items
openstackgerritEric Fried proposed openstack/nova-powervm master: WIP: update_provider_tree with device exposure
efried*really* don't understand why this caused a merge conflict, but whatever.20:07
*** esberglu has quit IRC22:02
*** esberglu has joined #openstack-powervm23:15
*** esberglu has quit IRC23:20
*** esberglu has joined #openstack-powervm23:40
*** esberglu has quit IRC23:45

Generated by 2.15.3 by Marius Gedminas - find it at!