<div dir="ltr"><div dir="ltr">Hi Ghanshyam,</div><div dir="ltr"><br></div><div dir="ltr"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Though I have concern over running those tests by default(making config options True by default), because it is not confirmed all cinder backends implements this functionality and it only works for nova libvirt driver. We need to keep config options default as False and Devstack/CI can make it True to run the tests. <br>
<br></blockquote><div><br></div><div>The discussion on the PTG was about whether we should run this on gate to actually break the CIs. Once that happens, vendors will have 3 options:</div><div><br></div><div>    #1: fix their drivers by properly implementing  volume_extend and run the positive tests</div><div>    #2: fix their drivers by reporting that they not support volume_extend and run the negative tests<br></div><div>    #3: disable volume extend tests at all (not recommendable), but this still give us a hint on whether the vendor supports this or not</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If this feature becomes mandatory functionality (or cinder say standard feature i think) to implement for every backends and it work with all nova driver also(in term of instance action events) then, we can enable this feature tests by default. But until then, we should keep them disable by default in Tempest but we can enable them on gate via Devstack (patch you mentioned) and test them daily on integrated-gate. <br></blockquote><div><br></div><div>Its not mandatory that the driver must implement online_extend, but if the driver does not support it, the driver should report as so.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Overall, I am ok with Devstack change to make these tests enable for every Cinder backends but we need to keep the config options false in Tempest. <br></blockquote><div><br></div><div>So, the outcome from the PTG was that we would first merge the tempest test and give time for vendors to get the drivers fixed. Then we would change it in devstack so we push vendor to fix their drivers in case they hadn't done that.</div><div><br></div><div>Erlon</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I will review those patch and leave comments on gerrit (i saw those patch introduce new config option than using the existing one)<br>
<br>
-gmann<br>
<br>
 > Please let us know if you have any question or concerns about it.<br>
 > Kind regards,Erlon_________________[1] <a href="https://review.openstack.org/#/c/572188/[2]" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/572188/[2]</a> <a href="https://review.openstack.org/#/c/578463/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/578463/</a> __________________________________________________________________________<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>
<br>
<br>
<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>
</blockquote></div></div>