<div dir="ltr">Hi,<div>We now have a working CI on below patches:</div><div><a href="https://review.openstack.org/#/c/187707/">https://review.openstack.org/#/c/187707/</a><br></div><div><a href="https://review.openstack.org/#/c/187853/">https://review.openstack.org/#/c/187853/</a><br></div><div><br></div><div>@jgriffith: we will sure start to give back to community.Thanks for pointing this out.</div><div><br></div><div>Regards</div><div>Nikesh</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 4, 2015 at 1:46 PM, Alex Meade <span dir="ltr"><<a href="mailto:mr.alex.meade@gmail.com" target="_blank">mr.alex.meade@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 dir="ltr">Agreed, I'd also like to mention that rebranded arrays may differ slightly in functionality as well so the CIs would need to run against a physical rebranded device. These differences also justify the need for letting rebranded drivers in.<span class="HOEnZb"><font color="#888888"><div><br></div><div>-Alex</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 4, 2015 at 4:41 PM, Mike Perez <span dir="ltr"><<a href="mailto:thingee@gmail.com" target="_blank">thingee@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sounds like the community would like CI's regardless, and I agree.<br>
<br>
Just because the driver code works for one backend solution, doesn't<br>
mean it's going to work with some other.<br>
<br>
Lets continue with code reviews with these patches only if they have a<br>
CI reporting, unless someone has a compelling reason we should not let<br>
any rebranded drivers in.<br>
<br>
--<br>
Mike Perez<br>
<div><div><br>
<br>
On Wed, Jun 3, 2015 at 10:32 AM, Mike Perez <<a href="mailto:thingee@gmail.com" target="_blank">thingee@gmail.com</a>> wrote:<br>
> There are a couple of cases [1][2] I'm seeing where new Cinder volume<br>
> drivers for Liberty are rebranding other volume drivers. This involves<br>
> inheriting off another volume driver's class(es) and providing some<br>
> config options to set the backend name, etc.<br>
><br>
> Two problems:<br>
><br>
> 1) There is a thought of no CI [3] is needed, since you're using<br>
> another vendor's driver code which does have a CI.<br>
><br>
> 2) IMO another way of satisfying a check mark of being OpenStack<br>
> supported and disappearing from the community.<br>
><br>
> What gain does OpenStack get from these kind of drivers?<br>
><br>
> Discuss.<br>
><br>
> [1] - <a href="https://review.openstack.org/#/c/187853/" target="_blank">https://review.openstack.org/#/c/187853/</a><br>
> [2] - <a href="https://review.openstack.org/#/c/187707/4" target="_blank">https://review.openstack.org/#/c/187707/4</a><br>
> [3] - <a href="https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers" target="_blank">https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers</a><br>
><br>
> --<br>
> Mike Perez<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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></div><br>__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>