<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 16, 2019 at 6:06 AM Ghanshyam Mann <<a href="mailto:gmann@ghanshyammann.com">gmann@ghanshyammann.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">---- On Mon, 16 Sep 2019 18:53:58 +0900 Ghanshyam Mann <<a href="mailto:gmann@ghanshyammann.com" target="_blank">gmann@ghanshyammann.com</a>> wrote ----<br>
 > Hello Everyone,<br>
 > <br>
 > As per discussion over ML, Tempest started the JSON schema strict validation for Volume  APIs response [1].<br>
 > Because it may affect the interop certification, it was explained to the Interop team as well as in the Board of Director meeting[2].<br>
 > <br>
 > In Train, Tempest started implementing the validation and found an API change where the new field was added in API response without versioning[3] (Cinder has API microversion mechanism). IMO, that was not the correct way to change the API and as per API-WG guidelines[4] any field added/modified/removed in API should be with microverison(means old versions/user should not be affected by that change) and must for API interoperability.<br>
 > <br>
 > With JSON schema validation, Tempest verifies the API interoperability recommended behaviour by API-WG.  But as per IRC conversion with cinder team, we have different opinion on API interoperability and how API should be changed under microversion mechanism.  I would like to have a conclusion on this so that Tempest can verify or leave the Volume API for strict validation. <br>
<br>
I found the same flow chart what Sean created in Nova about "when to bump microverison" in Cinder also which clearly say any addition to response need new microversion. <br>
- <a href="https://docs.openstack.org/cinder/latest/contributor/api_microversion_dev.html" rel="noreferrer" target="_blank">https://docs.openstack.org/cinder/latest/contributor/api_microversion_dev.html</a><br>
<br></blockquote><div><br></div><div>i would also expect any change in the request or response to result in a microversion bump as well.</div><div><br></div><div>peace o/</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-gmann<br>
<br>
 > <br>
 > [1] <a href="http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000358.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000358.html</a><br>
 > [2] <br>
 > - <a href="http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003652.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003652.html</a><br>
 > - <a href="http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003655.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003655.html</a><br>
 > [3] <a href="https://bugs.launchpad.net/tempest/+bug/1843762" rel="noreferrer" target="_blank">https://bugs.launchpad.net/tempest/+bug/1843762</a>   <a href="https://review.opendev.org/#/c/439461/" rel="noreferrer" target="_blank">https://review.opendev.org/#/c/439461/</a><br>
 > [4] <a href="https://specs.openstack.org/openstack/api-wg/guidelines/api_interoperability.html" rel="noreferrer" target="_blank">https://specs.openstack.org/openstack/api-wg/guidelines/api_interoperability.html</a><br>
 > <br>
 > -gmann<br>
 > <br>
<br>
<br>
</blockquote></div></div>