<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 11, 2016 at 4:23 AM, Ken'ichi Ohmichi <span dir="ltr"><<a href="mailto:ken1ohmichi@gmail.com" target="_blank">ken1ohmichi@gmail.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">Thanks for pointing this up, Jordan<br>
<br>
Before removing volume v1 API tests, it is nice to make the v2 API the<br>
default of Tempest scenario tests.<br>
Now the v1 and v2 is set as True on the default in the<br>
configuration[1], and the v1 API is used in the scenario like [2].<br>
So it is better to switch using v2 API on the default.<br></blockquote><div><a href="https://review.openstack.org/#/c/385050/" target="_blank">https://review.openstack.org/#<wbr>/c/385050/</a><br></div><div> </div><div>2016-10-10 11:37 GMT-07:00 Matt Riedemann <<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank">mriedem@linux.vnet.ibm.com</a>>:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>
><br>
> So make it conditional in Tempest via a config option, disable volume v1<br>
> tests by default for the integrated gate, and then add a new job that runs<br>
> only on cinder changes (and maybe only in the experimental queue) that<br>
> enables volume v1 tests. You could run it on cinder in the check/gate and<br>
> skip the job from running unless something in the v1 API path is changed,<br>
> there are examples of that in project-config.<br>
><br>
> Nova used to have the v2 API in tree and this was kind of the eventual path<br>
> to phasing out the Tempest testing on that code and got us to the point of<br>
> removing the v2 *code*.  The compute v2 API itself is still honored via the<br>
> v2.1 base microversion.<br></div></div></blockquote><div>Yeah that could work. But I'd rather we completely remove the Cinder v1 part in Tempest, I am tired of the "if cinder_v1 is True ... elif cinder_v2 is True ...", not to mention the name vs display name thing. Anyway, that's not my call.</div><div><br></div><div>I understand from Sean and Duncan reply that Cinder v1 is going to live on for a while, I'll see how we can react to this in Tempest.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>
><br>
> --<br>
><br>
> Thanks,<br>
><br>
> Matt Riedemann<br>
><br>
><br>
><br>
> ______________________________<wbr>______________________________<wbr>______________<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.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<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.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
</div></div></blockquote></div><br></div></div>

<br>
<a href="https://www.scality.com/backup/?utm_source=signatures&utm_medium=email&utm_campaign=backup2016" target="_blank"><img src="https://support.scality.com/Email_Footers/Current/Scality_Email_Footer.png"></a>