[Interop-wg] [dev] [cinder] [qa] Strict Validation for Volume API using JSON Schema
Ghanshyam Mann
gmann at ghanshyammann.com
Mon Dec 3 01:58:51 UTC 2018
---- On Sat, 01 Dec 2018 02:58:45 +0900 Mark Voelker <mvoelker at vmware.com> wrote ----
>
> > On Nov 29, 2018, at 9:28 PM, Matt Riedemann <mriedemos at gmail.com> wrote:
> >
> > On 11/29/2018 10:17 AM, Ghanshyam Mann wrote:
> >> - To improve the volume API testing to avoid the backward compatible changes. Sometime we accidentally change the API in backward incompatible way and strict validation with JSON schema help to block those.
> >
> > +1 this is very useful to avoid unintentionally breaking the API.
> >
> >> We want to hear from cinder and interop team about any impact of this change to them.
> >
> > I'm mostly interested in what the interop WG would do about this given it's a potentially breaking change for interop without changes to the guidelines. Would there be some sort of grace period for clouds to conform to the changes in tempest?
> >
>
> That’s more or less what eventually happened when we began enforcing strict validation on Nova a few years ago after considerable debate. Clouds that were compliant with the interop guidelines before the strict validation patch landed and started failing once it went in could apply for a waiver while they worked on removing or upstreaming the nonstandard stuff. For those not familiar, here’s the patch that created a waiver program:
>
> https://review.openstack.org/#/c/333067/
>
> Note that this expired with the 2017.01 Guideline:
>
> https://review.openstack.org/#/c/512447/
>
> While not everyone was totally happy with the solution, it seemed to work out as a middle ground solution that helped get clouds on a better path in the end. I think we’ll discuss whether we’d need to do something like this again here. I’d love to hear:
>
> 1.) If anyone knows of clouds/products that would be fail interop testing because of this. Not looking to name and shame, just to get an idea of whether or not we have a concrete problem and how big it is.
>
> 2.) Opinions on how the waiver program went last time and whether the rest of the community feels like it’s something we should consider again.
>
> Personally I’m supportive of the general notion of improving API interoperability here…as usual it’s figuring out the mechanics of the transition that take a little figuring. =)
Thanks Mark for response. I think point 1 is important, it is good to get the list of clouds or failure due to this this strict validation change. And accordingly, we can wait on Tempest side to merge those changes for this cycle (but personally I do not want to delay that if everything is fine), so that we can avoid the immediate failure of interop program.
-gmann
>
> At Your Service,
>
> Mark T. Voelker
>
>
> > --
> >
> > Thanks,
> >
> > Matt
> >
> > _______________________________________________
> > Interop-wg mailing list
> > Interop-wg at lists.openstack.org
> > https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.openstack.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Finterop-wg&data=02%7C01%7Cmvoelker%40vmware.com%7C82a07fe28afe488c2eea08d6566b9734%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C636791417437738014&sdata=lEx%2BbbTVzC%2FRC7ebmARDrFhfMsToM7Rwx8EKYtE7iFM%3D&reserved=0
>
>
More information about the openstack-discuss
mailing list