<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 16, 2017 at 1:51 AM, Tristan Cacqueray <span dir="ltr"><<a href="mailto:tdecacqu@redhat.com" target="_blank">tdecacqu@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On June 14, 2017 1:10 pm, James E. Blair wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Tristan Cacqueray <<a href="mailto:tdecacqu@redhat.com" target="_blank">tdecacqu@redhat.com</a>> writes:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
With the nodepool-drivers[0] spec approved, I started to hack a quick<br>
implementation[1]. Well I am not very familiar with the nodepool/zookeeper<br>
architecture, thus this implementation may very well be missing important<br>
bits... The primary goal is to be able to run ZuulV3 with static nodes,<br>
comments and feedbacks are most welcome.<br>
</blockquote>
<br>
I've taken a general look and I think this is heading in the right<br>
direction.  We should ask David Shrewsbury to look at it when he gets a<br>
chance, and Tobias as well when he's back.  Thanks!<br>
<br>
</blockquote></span>
Thanks! The first three changes mostly move code to the driver directory<br>
without changing the logic which seems like a sane thing to do before<br>
changing the interface. Since this is a pain to rebase, those should be the<br>
priority:<br>
* <a href="https://review.openstack.org/468748" rel="noreferrer" target="_blank">https://review.openstack.org/4<wbr>68748</a> : generic request handler<br>
* <a href="https://review.openstack.org/468749" rel="noreferrer" target="_blank">https://review.openstack.org/4<wbr>68749</a> : generic provider manager<br>
* <a href="https://review.openstack.org/468750" rel="noreferrer" target="_blank">https://review.openstack.org/4<wbr>68750</a> : move openstack bits to its own driver<br></blockquote><div><br></div><div><br></div><div>Overall, these three seem ok after studying them for a bit. And tests pass, so</div><div>that's a plus.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
A follow-up effort would be to also move openstack driver tests to their own<br>
files and the provider configuration to the driver module.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Moreover, assuming this isn't too off-track, I'd like to propose an<br>
OpenContainer and a libvirt driver to diversify Test environment.<br>
</blockquote>
<br>
I think the most important thing is the static node driver -- that's<br>
part of the original scope for Zuul v3, and we need it for functional<br>
parity with v2.<br>
<br>
An OpenContainer driver sounds fine to me, but I'm reluctant to add a<br>
libvirt driver at the moment -- there is a lot of potential overlap with<br>
OpenStack, as well as other potential drivers such as linch-pin.  Maybe<br>
there are some compelling reasons to do so, but I'd rather defer that<br>
for a while until we establish some guidelines around in-tree drivers.<br>
<br>
Since it's a scope expansion, we should consider anything beyond the<br>
static driver to be a lower priority while we work to get Zuul v3<br>
finished.<br>
<br>
</blockquote></span>
Indeed, having static nodes is the primary goal, the extra drivers are<br>
mainly to exercise the interface for now. It's fine if they are not<br>
accepted in-tree, as long as we design a common interface.<br>
<br>
Cheers,<br>
-Tristan<br>
<br>______________________________<wbr>_________________<br>
OpenStack-Infra mailing list<br>
<a href="mailto:OpenStack-Infra@lists.openstack.org">OpenStack-Infra@lists.<wbr>openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-infra</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>David Shrewsbury (Shrews)<br></div></div></div>
</div></div>