<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 11, 2014 at 2:18 AM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On Thu, Sep 11, 2014 at 09:23:34AM +1000, Michael Still wrote:<br>
> On Thu, Sep 11, 2014 at 8:11 AM, Jay Pipes <<a href="mailto:jaypipes@gmail.com">jaypipes@gmail.com</a>> wrote:<br>
><br>
> > a) Sorting out the common code is already accounted for in Dan B's original<br>
> > proposal -- it's a prerequisite for the split.<br>
><br>
> Its a big prerequisite though. I think we're talking about a release<br>
> worth of work to get that right. I don't object to us doing that work,<br>
> but I think we need to be honest about how long its going to take. It<br>
> will also make the core of nova less agile, as we'll find it hard to<br>
> change the hypervisor driver interface over time. Do we really think<br>
> its ready to be stable?<br>
<br>
Yes, in my proposal I explicitly said we'd need to have Kilo<br>
for all the prep work to clean up the virt API, before only<br>
doing the split in Lxxxxx.<br>
<br>
The actual nova/virt/driver.py has been more stable over the<br>
past few releases than I thought it would be. In terms of APIs<br>
we're not really modified existing APIs, mostly added new ones.<br>
Where we did modify existing APIs, we could have easily taken<br>
the approach of adding a new API in parallel and deprecating<br>
the old entry point to maintain compat.<br>
<br>
The big change which isn't visible directly is the conversion<br>
of internal nova code to use objects. Finishing this conversion<br>
is clearly a pre-requisite to any such split, since we'd need<br>
to make sure all data passed into the nova virt APIs as parameters<br>
is stable & well defined.<br>
<br>
> As an alternative approach...<br>
><br>
> What if we pushed most of the code for a driver into a library?<br>
> Imagine a library which controls the low level operations of a<br>
> hypervisor -- create a vm, attach a NIC, etc. Then the driver would<br>
> become a shim around that which was relatively thin, but owned the<br>
> interface into the nova core. The driver handles the nova specific<br>
> things like knowing how to create a config drive, or how to<br>
> orchestrate with cinder, but hands over all the hypervisor operations<br>
> to the library. If we found a bug in the library we just pin our<br>
> dependancy on the version we know works whilst we fix things.<br>
><br>
> In fact, the driver inside nova could be a relatively generic "library<br>
> driver", and we could have multiple implementations of the library,<br>
> one for each hypervisor.<br>
<br>
I don't think that particularly solves the problem, particularly<br>
the ones you are most concerned about above of API stability. The<br>
naive impl of any "library" for the virt driver would pretty much<br>
mirror the nova virt API. The virt driver impls would thus have to<br>
do the job of taking the Nova objects passed in as parameters and<br>
turning them into something "stable" to pass to the library. Except<br>
now instead of us only having to figure out a stable API in one<br>
place, every single driver has to reinvent the wheel defining their<br>
own stable interface & objects. I'd also be concerned that ongoing<br>
work on drivers is still going to require alot of patches to Nova<br>
to update the shims all the time, so we're still going to contend<br>
on resource fairly highly.<br>
<br>
> > b) The conflict Dan is speaking of is around the current situation where we<br>
> > have a limited core review team bandwidth and we have to pick and choose<br>
> > which virt driver-specific features we will review. This leads to bad<br>
> > feelings and conflict.<br>
><br>
> The way this worked in the past is we had cores who were subject<br>
> matter experts in various parts of the code -- there is a clear set of<br>
> cores who "get" xen or libivrt for example and I feel like those<br>
> drivers get reasonable review times. What's happened though is that<br>
> we've added a bunch of drivers without adding subject matter experts<br>
> to core to cover those drivers. Those newer drivers therefore have a<br>
> harder time getting things reviewed and approved.<br>
<br>
FYI, for Juno at least I really don't consider that even the libvirt<br>
driver got acceptable review times in any sense. The pain of waiting<br>
for reviews in libvirt code I've submitted this cycle is what prompted<br>
me to start this thread. All the virt drivers are suffering way more<br>
than they should be, but those without core team representation suffer<br></blockquote><div><br></div><div><div>Can't you replace the word 'libvirt code' with 'nova code' and this would still be true? Do you think landing virt driver code is harder then landing non virt driver code? If so do you have any numbers to back this up? </div><div><br></div><div>If the issue here is 'landing code in nova is too painful', then we should discuss solving that more generalized issue first, and maybe we conclude that pulling out the virt drivers gets us the most bang for our buck. But unless we have that more general discussion, saying the right fix for that is to spend a large amount of time working specifically on virt driver related issues seems premature.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
to an even greater degree. And this is ignoring the point Jay & I<br>
were making about how the use of a single team means that there is<br>
always contention for feature approval, so much work gets cut right<br>
at the start even if maintainers of that area felt it was valuable<br>
and worth taking.<br>
<br>
> > c) It's the impact to the CI and testing load that I see being the biggest<br>
> > benefit to the split-out driver repos. Patches proposed to the XenAPI driver<br>
> > shouldn't have the Hyper-V CI tests run against the patch. Likewise, running<br>
> > libvirt unit tests in the VMWare driver repo doesn't make a whole lot of<br>
> > sense, and all of these tests add a not-insignificant load to the overall<br>
> > upstream and external CI systems. The long wait time for tests to come back<br>
> > means contributors get frustrated, since many reviewers tend to wait until<br>
> > Jenkins returns some result before they review. All of this leads to<br>
> > increased conflict that would be somewhat ameliorated by having separate<br>
> > code repos for the virt drivers.<br>
><br>
> It is already possible to filter CI runs to specific paths in the<br>
> code. We just didn't choose to do that for policy reasons. We could<br>
> change that right now with a trivial tweak to each CI system's zuul<br>
> config.<br>
<br>
We have to jump through far more hoops to do so, even as developers<br>
running things locally. eg want to run pep8 locally to test your<br>
work ? You have to wait 3 minutes while it checks the entire of<br>
the nova codebase. So we had to invent a mode where it only checks<br>
the files in the current GIT HEAD. Likewise for unit tests - if you<br>
invoke them you have to pass args to filter to just the area of the<br>
repo you are working on. These kind of problems simply goes away<br>
completely if we have separate repos without having to do special<br>
setup tasks. Smaller modules would be far less daunting for new<br>
contributors looking to get involved in Nova development too which<br>
I think is an important factor<br>
<br>
Regards,<br>
Daniel<br>
<span class=""><font color="#888888">--<br>
|: <a href="http://berrange.com" target="_blank">http://berrange.com</a> -o- <a href="http://www.flickr.com/photos/dberrange/" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" target="_blank">http://libvirt.org</a> -o- <a href="http://virt-manager.org" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" target="_blank">http://autobuild.org</a> -o- <a href="http://search.cpan.org/~danberr/" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" target="_blank">http://entangle-photo.org</a> -o- <a href="http://live.gnome.org/gtk-vnc" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</font></span><div class=""><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>