<p dir="ltr">Right, I was just pointing out that solving this in os-vif isn't enough in this particular case since neutron can't use it so it's slightly different than the os-brick situation. We have to reason about what two different code paths will do. </p>
<div class="gmail_extra"><br><div class="gmail_quote">On Aug 29, 2016 6:41 AM, "Sean Dague" <<a href="mailto:sean@dague.net">sean@dague.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08/29/2016 08:29 AM, Kevin Benton wrote:<br>
> Sort of. The neutron agent code doesn't use os-vif because the os-vif<br>
> devs indicated that neutron's vif plugging code wasn't a use case they<br>
> cared about [1].<br>
><br>
> So if we did generalize os-vif to work with the neutron agents then it<br>
> would be two calling the same locked code. But at this point it's just<br>
> two versions of similar logic trying to do the same thing.<br>
><br>
> 1. <a href="https://review.openstack.org/#/c/284209/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/284209/</a><br>
<br>
Calling the same locked code wouldn't help, these are different services<br>
that in *almost* all real deployments are running under different user<br>
ids. Which means shared locks between them are basically not possible.<br>
<br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
<a href="http://dague.net" rel="noreferrer" target="_blank">http://dague.net</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></div>