<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 24, 2016 at 6:06 PM, Slade Baumann <span dir="ltr"><<a href="mailto:baumann@us.ibm.com" target="_blank">baumann@us.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font face="Verdana,Arial,Helvetica,sans-serif" size="2">I am attempting to disable clone tests in tempest as they aren't<br>functioning in NFS. But the tests test_volumes_clone.py and<br>test_volumes_clone_negative.py don't have the "clone" feature<br>toggle in them. I thought it obvious that if clone is disabled<br>in tempest, the tests that simply clone should be disabled.<br><br>So I put up a bug and fix for it, but have been talking with<br>Jordan Pittier and he suggested I come to the mailing list to<br>get this figured out. <br><br>I'm not asking for reviews, unless you want to give them.<br>I'm simply asking if this is the right way to go about this<br>or if there is something else I need to do to get this into<br>Tempest.<br><br>Here are the bug and fix:<br><a href="https://bugs.launchpad.net/tempest/+bug/1615770" target="_blank">https://bugs.launchpad.net/<wbr>tempest/+bug/1615770</a><br><a href="https://review.openstack.org/#/c/358813/" target="_blank">https://review.openstack.org/#<wbr>/c/358813/</a><br><br>I would appreciate any suggestion or direction in this problem.<br><br>For extra reference, the clone toggle flag was added here:<br><a href="https://bugs.launchpad.net/tempest/+bug/1488274" target="_blank">https://bugs.launchpad.net/<wbr>tempest/+bug/1488274</a></font><br>


<br></blockquote><div>Hi, </div><div>Thanks for starting this thread. My point about this patch is, as "volume clone" is part of the core requirements [1] every Cinder drive must support, I don't see a need for a feature flag. The feature flag already exists, but that doesn't mean we should encourage its usage.</div><div><br></div><div>Now, if this really helps the NFS driver (although I don"t know why we couldn't support clone with NFS)... I don't have a strong opinion on this patch.</div><div><br></div><div>I -1ed the patch for consistency: I agree that there should be a minimum set of features expected from a Cinder driver.</div><div><br></div><div>[1] <a href="http://docs.openstack.org/developer/cinder/devref/drivers.html#core-functionality">http://docs.openstack.org/developer/cinder/devref/drivers.html#core-functionality</a></div><div><br></div><div>Cheers,</div><div>Jordan </div></div></div></div>

<br>
<a href="http://bit.ly/2aKbaTu" target="_blank"><img src="https://support.scality.com/Email_Footers/Current/Scality_Email_Footer_RING6.png"></a>