<div dir="ltr"><div>Some of the stuff like '802.1qbh' isn't particularly vendor specific so I'm not sure who will host it and a repo just for that seems like a bit much. Should we just bite the bullet and convert them in the nova tree or put them in os-vif?<br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 13, 2017 at 7:26 AM, Stephen Finucane <span dir="ltr"><<a href="mailto:sfinucan@redhat.com" target="_blank">sfinucan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">os-vif has been integrated into nova since the newton cycle. With the<br>
integration of os-vif, the expectation is that all the old, non-os-vif<br>
plugging/unplugging code found in [1] will be replaced by code that harnesses<br>
os-vif plugins [2]. This has happened for a few of the VIF types, and newer<br>
VIFs are being added in this manner [3]. However, there are quite a few VIFs<br>
that are still using the legacy path, and I think it's about time we started<br>
moving things forward. Doing so allows us to continue to progress on passing<br>
os-vif objects from neutron and remove the large swathes of legacy code still<br>
found in nova.<br>
<br>
I've opened a bug against networking-bigswitch [4] for one of these VIF types,<br>
IVS, and I'm thinking I'll do the same for a lot of the other VIF types where I<br>
can find definite vendors. Is there anything else we can do though? At some<br>
point we're going to have to just start deleting code and I'd like to avoid<br>
leaving operators in the lurch.<br>
<br>
Cheers,<br>
Stephen<br>
<br>
[1] <a href="https://github.com/openstack/nova/blob/6205a3f8c/nova/virt/libvirt/vif.py#L
599-L764" rel="noreferrer" target="_blank">https://github.com/openstack/<wbr>nova/blob/6205a3f8c/nova/virt/<wbr>libvirt/vif.py#L<br>
599-L764</a><br>
[2] <a href="https://github.com/openstack/nova/blob/6205a3f8c/nova/network/os_vif_util.p
y#L346-L403" rel="noreferrer" target="_blank">https://github.com/openstack/<wbr>nova/blob/6205a3f8c/nova/<wbr>network/os_vif_util.p<br>
y#L346-L403</a><br>
[3] <a href="https://github.com/Juniper/contrail-nova-vif-driver" rel="noreferrer" target="_blank">https://github.com/Juniper/<wbr>contrail-nova-vif-driver</a><br>
[4] <a href="https://bugs.launchpad.net/networking-bigswitch/+bug/1704129" rel="noreferrer" target="_blank">https://bugs.launchpad.net/<wbr>networking-bigswitch/+bug/<wbr>1704129</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</blockquote></div><br></div>