<p dir="ltr">That is a semantic change to the api that will break anybody who has tooling expecting the current behavior. Since there are perfectly sensible uses of the current behavior, that is not a good thing.</p>
<div class="gmail_quote">On 10 Jul 2015 07:33, "hao wang" <<a href="mailto:sxmatch1986@gmail.com">sxmatch1986@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><font face="arial, helvetica, sans-serif">Cinder<span style="color:rgb(0,0,0)"> now doesn't check the existing resource when user lower the quota. </span></font><div><font color="#000000" face="arial, helvetica, sans-serif">It's reasonable for admin can adjust the quota limit to lower level than current usage.</font></div><div><font color="#000000" face="arial, helvetica, sans-serif">But it also bring confusion that I have received to end user, they saw the current usage</font></div><div><font color="#000000" face="arial, helvetica, sans-serif">was more than limit, but they can't create resources any more.</font></div><div><font color="#000000" face="arial, helvetica, sans-serif"><br></font></div><div><font color="#000000" face="arial, helvetica, sans-serif">So there have been 'bug' reported[1] and code patch[2] committed, I knew it may be </font></div><div><span style="line-height:18px"><font face="arial, helvetica, sans-serif" color="#000000">inappropriate as 'bug fix', but just want to </font></span><font face="arial, helvetica, sans-serif" color="#000000"><span style="line-height:18px">optimize this API of updating quota.</span></font><br></div><div><font face="arial, helvetica, sans-serif" color="#000000"><span style="line-height:18px"><br></span></font></div><div><font color="#000000" face="arial, helvetica, sans-serif"><span style="line-height:18px">We are proposing to add an option argument which is named 'force' in request body.</span></font></div><div><font color="#000000" face="arial, helvetica, sans-serif"><span style="line-height:18px">Of course the default value is True that means admin can adjust the quota lower then</span></font></div><div><font color="#000000" face="arial, helvetica, sans-serif"><span style="line-height:18px">current usage as same as what we did now. When the force is False, that will occur </span></font></div><div><span style="color:rgb(0,0,0)"><font face="arial, helvetica, sans-serif">a Validation and return 400 Bad Request if </font></span><span style="color:rgb(0,0,0)"><font face="arial, helvetica, sans-serif">the update value is lower than current usage.</font></span></div><div><span style="color:rgb(0,0,0)"><font face="arial, helvetica, sans-serif"><br></font></span></div><div><font color="#000000" face="arial, helvetica, sans-serif">I wonder to know folks' opinions and suggestions about this change to see if this is value to merge this patch.</font></div><div><br></div><div><font color="#000000" face="arial, helvetica, sans-serif">[1]<a href="https://bugs.launchpad.net/neutron/+bug/1304234" target="_blank">https://bugs.launchpad.net/neutron/+bug/1304234</a></font></div><div><font color="#000000" face="arial, helvetica, sans-serif">[2]<a href="https://review.openstack.org/#/c/197938/" target="_blank">https://review.openstack.org/#/c/197938/</a></font></div><div><font color="#000000" face="arial, helvetica, sans-serif"><br></font></div><div><font color="#000000" face="arial, helvetica, sans-serif">Thanks~</font></div><div><div><br></div>-- <br><div><pre>Best Wishes For You!</pre></div>
</div></div>
<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>