<html><body>
<p><font size="2" face="sans-serif">I'd like you provide your comments on this patch.</font><br>
<br>
<font size="2" face="sans-serif">I prefer to keep the check in the patch and add "--force" to make it possible to change the quota like before my patch.</font><br>
<br>
<font size="2" face="sans-serif">Any thought?</font><br>
<br>
<font size="2" face="sans-serif"><br>
--------------------------------------------------<br>
Best regard, <br>
David Geng<br>
--------------------------------------------------</font><br>
<font size="1" color="#800080" face="sans-serif">----- Forwarded by Jian Hua Geng/China/IBM</font><font size="1" color="#800080" face="sans-serif"> on 04/02/2013 10:38 PM</font><font size="1" color="#800080" face="sans-serif"> -----</font><br>
<br>

<table width="100%" border="0" cellspacing="0" cellpadding="0">
<tr valign="top"><td style="background-image:url(cid:1__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com); background-repeat: no-repeat; " width="40%">
<ul style="padding-left: 72pt"><font size="1" face="sans-serif"><b>"Nikola Dipanov (Code Review)" <review@openstack.org></b></font><font size="1" face="sans-serif"> </font>
<p><font size="1" face="sans-serif">04/02/2013 09:08 PM</font>
<table border="1">
<tr valign="top"><td width="168" bgcolor="#FFFFFF"><div align="center"><font size="1" face="sans-serif">Please respond to<br>
ndipanov@redhat.com</font></div></td></tr>
</table>
</ul>
</td><td width="60%">
<table width="100%" border="0" cellspacing="0" cellpadding="0">
<tr valign="top"><td width="1%"><img width="58" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""><br>
<div align="right"><font size="1" face="sans-serif">To</font></div></td><td width="100%"><img width="1" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""><br>

<ul style="padding-left: 7pt"><font size="1" face="sans-serif">gengjh@cn.ibm.com</font></ul>
</td></tr>

<tr valign="top"><td width="1%"><img width="58" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""><br>
<div align="right"><font size="1" face="sans-serif">cc</font></div></td><td width="100%"><img width="1" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""><br>

<ul style="padding-left: 7pt"><font size="1" face="sans-serif">Vish Ishaya <vishvananda@gmail.com>, "Kevin L. Mitchell" <kevin.mitchell@rackspace.com>, "Davanum Srinivas (dims)" <davanum@gmail.com></font></ul>
</td></tr>

<tr valign="top"><td width="1%"><img width="58" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""><br>
<div align="right"><font size="1" face="sans-serif">Subject</font></div></td><td width="100%"><img width="1" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""><br>

<ul style="padding-left: 7pt"><font size="1" face="sans-serif">Change in openstack/nova[master]: Enhance the validation of the quotas update</font></ul>
</td></tr>
</table>

<table border="0" cellspacing="0" cellpadding="0">
<tr valign="top"><td width="58"><img width="1" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""></td><td width="336"><img width="1" height="1" src="cid:2__=C7BBF1D2DFC3EE998f9e8a93df938@cn.ibm.com" border="0" alt=""></td></tr>
</table>
</td></tr>
</table>
<br>
<tt><font size="2">Nikola Dipanov has posted comments on this change.<br>
<br>
Change subject: Enhance the validation of the quotas update<br>
......................................................................<br>
<br>
<br>
Patch Set 2:<br>
<br>
Ideally - we should be able to notify users that their quota has been decreased and that they will not be able to do what they think they might be able to do without their failing jobs telling them that.<br>
<br>
This may not need to be automated, but would be good to let the admin know that certain tenants/users will be immediately blocked by this action. Some ideas:<br>
<br>
* maybe leave this as is in the patch, but have the api return all the users this fails for, and add a --force to still make it possible to change it like before the patch.<br>
<br>
* Leave it as is now, but have the api call return the list of immediately affected users.<br>
<br>
All of these would have to wait for H tho.<br>
<br>
These are only a few options with pros and cons. I'd bring this up on the dev list and see what the community thinks.<br>
<br>
--<br>
To view, visit </font></tt><tt><font size="2"><a href="https://review.openstack.org/25887">https://review.openstack.org/25887</a></font></tt><tt><font size="2"><br>
To unsubscribe, visit </font></tt><tt><font size="2"><a href="https://review.openstack.org/settings">https://review.openstack.org/settings</a></font></tt><tt><font size="2"><br>
<br>
Gerrit-MessageType: comment<br>
Gerrit-Change-Id: Iba3cee0f0d92cf2e6d64bc83830b0091992d1ee9<br>
Gerrit-PatchSet: 2<br>
Gerrit-Project: openstack/nova<br>
Gerrit-Branch: master<br>
Gerrit-Owner: David Geng <gengjh@cn.ibm.com><br>
Gerrit-Reviewer: Davanum Srinivas (dims) <davanum@gmail.com><br>
Gerrit-Reviewer: David Geng <gengjh@cn.ibm.com><br>
Gerrit-Reviewer: Jenkins<br>
Gerrit-Reviewer: Kevin L. Mitchell <kevin.mitchell@rackspace.com><br>
Gerrit-Reviewer: Nikola Dipanov <ndipanov@redhat.com><br>
Gerrit-Reviewer: SmokeStack<br>
Gerrit-Reviewer: Vish Ishaya <vishvananda@gmail.com><br>
<br>
</font></tt></body></html>