<div dir="ltr"><div>Thanks a lot Matt, Jay and Dan for your reactivity and your time.<br><br></div>Édouard.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 21, 2018 at 9:50 PM, Matt Riedemann <span dir="ltr"><<a href="mailto:mriedemos@gmail.com" target="_blank">mriedemos@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 2/21/2018 4:30 AM, Édouard Thuleau wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
Hi Seán, Michael,<br>
<br>
Since patch [1] moved Contrail VIF plugging under privsep, Nova fails to plug TAP on the Contrail software switch (named vrouter) [2]. I proposed a fix in the beginning of the year [3] but it still pending approval even it got a couple of +1 and no negative feedback. It's why I'm writing that email to get your attention.<br>
That issue appeared during the Queens development cycle and we need to fix that before it was released (hope we are not to late).<br>
Contrail already started to move on os-vif driver [4]. A first VIF type driver is there for DPDK case [5], we plan to do the same for the TAP case in the R release and remove the Nova VIF plugging code for the vrouter.<br>
<br>
[1] <a href="https://review.openstack.org/#/c/515916/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/515916/</a><br>
[2] <a href="https://bugs.launchpad.net/nova/+bug/1742963" rel="noreferrer" target="_blank">https://bugs.launchpad.net/nov<wbr>a/+bug/1742963</a><br>
[3] <a href="https://review.openstack.org/#/c/533212/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/533212/</a><br>
[4] <a href="https://github.com/Juniper/contrail-nova-vif-driver" rel="noreferrer" target="_blank">https://github.com/Juniper/con<wbr>trail-nova-vif-driver</a><br>
[5] <a href="https://review.openstack.org/#/c/441183/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/441183/</a><br>
<br>
Regards,<br>
Édouard.<br>
<br>
<br></div></div>
______________________________<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.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
<br>
</blockquote>
<br>
Approved the change on master and working on the backport to stable/queens. We'll be cutting an RC3 tomorrow so I'll make sure this gets into that.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt<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.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
</font></span></blockquote></div><br></div>