<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 1, 2014 at 1:32 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi David,<br>
<br>
2014-05-01 5:44 GMT+09:00 David Kranz <<a href="mailto:dkranz@redhat.com">dkranz@redhat.com</a>>:<br>
<div class="">> There have been a lot of patches that add the validation of response dicts.<br>
> We need a policy on whether this is required or not. For example, this patch<br>
><br>
> <a href="https://review.openstack.org/#/c/87438/5" target="_blank">https://review.openstack.org/#/c/87438/5</a><br>
><br>
> is for the equivalent of 'cinder service-list' and is a basically a copy of<br>
> the nova test which now does the validation. So two questions:<br>
><br>
> Is cinder going to do this kind of checking?<br>
> If so, should new tests be required to do it on submission?<br>
<br>
</div>I'm not sure someone will add the similar validation, which we are adding to<br>
Nova API tests, to Cinder API tests also. but it would be nice for Cinder and<br>
Tempest. The validation can be applied to the other projects(Cinder, etc)<br>
easily because the base framework is implemented in common rest client<br>
of Tempest.<br>
<br>
When adding new tests like <a href="https://review.openstack.org/#/c/87438" target="_blank">https://review.openstack.org/#/c/87438</a> , I don't<br>
have strong opinion for including the validation also. These schemas will be<br>
large sometimes and the combination in the same patch would make reviews<br>
difficult. In current Nova API test implementations, we are separating them<br>
into different patches.<br>
<br></blockquote><div><br></div><div>Separating the schema part into a separate dependent patch probably makes sense when they large but I would like to see us ratchet the requirement to have schema validation for the cinder api as well too. <br>
<br></div><div>Chris<br></div><div><br><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Ken'ichi Ohmichi<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>