<div dir="ltr"><div>Yep, the way we removed the validation is not good long term solution (IMO) because we still requesting the schema for unvalidated_model and I am not sure why do we need it.<br></div><div>I will create a spec about it soon so we can discuss it in more details.<br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Best regards,<div>Kairat Kushaev</div></div></div></div></div></div>
<br><div class="gmail_quote">On Thu, Oct 1, 2015 at 2:44 PM,  <span dir="ltr"><<a href="mailto:stuart.mclaren@hp.com" target="_blank">stuart.mclaren@hp.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
We've been taking validation out as issues have been reported (it was<br>
removed from image-list recently for example).<br>
<br>
Removing across the board probably does make sense.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
<br>
Agree with you. That's why I am asking about reasoning. Perhaps, we need to<br>
realize how to get rid of this in glanceclient.<br>
<br>
Best regards,<br>
Kairat Kushaev<br>
<br>
On Wed, Sep 30, 2015 at 7:04 PM, Jay Pipes <<a href="mailto:jaypipes@gmail.com" target="_blank">jaypipes@gmail.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 09/30/2015 09:31 AM, Kairat Kushaev wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi All,<br>
In short terms, I am wondering why we are validating responses from<br>
server when we are doing<br>
image-show, image-list, member-list, metadef-namespace-show and other<br>
read-only requests.<br>
<br>
AFAIK, we are building warlock models when receiving responses from<br>
server (see [0]). Each model requires schema to be fetched from glance<br>
server. It means that each time we are doing image-show, image-list,<br>
image-create, member-list and others we are requesting schema from the<br>
server. AFAIU, we are using models to dynamically validate that object<br>
is in accordance with schema but is it the case when glance receives<br>
responses from the server?<br>
<br>
Could somebody please explain me the reasoning of this implementation?<br>
Am I missed some usage cases when validation is required for server<br>
responses?<br>
<br>
I also noticed that we already faced some issues with such<br>
implementation that leads to "mocking" validation([1][2]).<br>
<br>
</blockquote>
<br>
The validation should not be done for responses, only ever requests (and<br>
it's unclear that there is value in doing this on the client side at all,<br>
IMHO).<br>
<br>
-jay<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>
<br>
</blockquote></div></div>
-------------- next part --------------<br>
An HTML attachment was scrubbed...<br>
URL: <<a href="http://lists.openstack.org/pipermail/openstack-dev/attachments/20150930/5b5dba74/attachment-0001.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-dev/attachments/20150930/5b5dba74/attachment-0001.html</a>><br>
</blockquote><div class="HOEnZb"><div class="h5">
<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>
</div></div></blockquote></div><br></div>