<div dir="ltr"><font face="arial, helvetica, sans-serif">Hi, Mike </font><div><font face="arial, helvetica, sans-serif"><br></font><div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)">I'm not sure we really don't need any change about this feature. At least, some end users I faced to think there should be changed....</span></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)"><br></span></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)">IMHO, there is a main problem that some users whom I faced to can't understand: What's the purpose that admin reduce quota lowner than existing usage? Limit user to can't create any resources any more? But why reduce quota just equal the current usage, it has same function. Make user to delete their resources lower than the new limit line? It's weak if user don't want to do that deletion and also bring some confusion to other users that I have mentioned. </span></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)"><br></span></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)">I understood there may be 100 reasons to show me why admin can reduce the quota lower than usage, and I don't want to object them too. But I hope this change can bring some new usage to update quota: 1. When admin use client(could be third party) to update the quota limit, they should check quota usage first as winston mentioned, if they don't or forget, anyway, they will change failed if quota is lower than usage, since we give the ability to cinder it will stop them to do that thing and make admin back to check quota usage. 2. If admin know what they are doing and just need to reduce the limit lower for some reason, fine, take the option argument '--force' or '--skip_validation' to update the quota. </span></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)"><br></span></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)">In personally, I felt this routine may be more improvement and little confusion with it. I knew Eric said that of course we can implement this purpose by using current APIs, it's a alternatives, but it depends on the application which is top on cinder I think, and is hard to have consistent.</span></font></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-07-11 7:24 GMT+08:00 Mike Perez <span dir="ltr"><<a href="mailto:thingee@gmail.com" target="_blank">thingee@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 12:30 Jul 10, hao wang wrote:<br>
> Cinder now doesn't check the existing resource when user lower the quota.<br>
> It's reasonable for admin can adjust the quota limit to lower level than<br>
> current usage.<br>
> But it also bring confusion that I have received to end user, they saw the<br>
> current usage<br>
> was more than limit, but they can't create resources any more.<br>
><br>
> So there have been 'bug' reported[1] and code patch[2] committed, I knew it<br>
> may be<br>
> inappropriate as 'bug fix', but just want to optimize this API of updating<br>
> quota.<br>
><br>
> We are proposing to add an option argument which is named 'force' in<br>
> request body.<br>
> Of course the default value is True that means admin can adjust the quota<br>
> lower then<br>
> current usage as same as what we did now. When the force is False, that<br>
> will occur<br>
> a Validation and return 400 Bad Request if the update value is lower than<br>
> current usage.<br>
><br>
> I wonder to know folks' opinions and suggestions about this change to see<br>
> if this is value to merge this patch.<br>
<br>
</span>Based on the feedback received in the bug and review, it seems like there is<br>
a clear consensus that people don't want this, even if it can be bypassed with<br>
a force option.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Mike Perez<br>
</font></span><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><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><pre>Best Wishes For You!</pre></div>
</div>