<div dir="ltr">On 10 December 2014 at 01:31, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
</span>So the problem of Nova review bandwidth is a constant problem across all<br>
areas of the code. We need to solve this problem for the team as a whole<br>
in a much broader fashion than just for people writing VIF drivers. The<br>
VIF drivers are really small pieces of code that should be straightforward<br>
to review & get merged in any release cycle in which they are proposed.<br>
I think we need to make sure that we focus our energy on doing this and<br>
not ignoring the problem by breaking stuff off out of tree.<br>
</blockquote><div><br></div><div>The problem is that we effectively prevent running an out of tree Neutron driver (which *is* perfectly legitimate) if it uses a VIF plugging mechanism that isn't in Nova, as we can't use out of tree code and we won't accept in code ones for out of tree drivers.  This will get more confusing as *all* of the Neutron drivers and plugins move out of the tree, as that constraint becomes essentially arbitrary.<br><br></div><div>Your issue is one of testing.  Is there any way we could set up a better testing framework for VIF drivers where Nova interacts with something to test the plugging mechanism actually passes traffic?  I don't believe there's any specific limitation on it being *Neutron* that uses the plugging interaction.<br></div><div>-- <br></div><div>Ian.<br></div></div></div></div>