<div dir="ltr">Thanks for address the issues. About the bad state for fixed_ips, floating_ips, i think we could make the user_id column=NULL when creating the quota usage and reservation, so the usages for fixed_ips and floating_ips will be  synced within the project.<div style>
Does this make sense?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/8/20 Andrew Laski <span dir="ltr"><<a href="mailto:andrew.laski@rackspace.com" target="_blank">andrew.laski@rackspace.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The patch in question (<a href="https://review.openstack.org/#/c/28232/24" target="_blank">https://review.openstack.org/<u></u>#/c/28232/24</a>) adds the ability to track quota usage on a per user basis within a project.  I have run into two issues with it so far: the db migration is incomplete and leaves the data in a bad state, and the sync methods used during quota reservations no longer work for fixed_ips, floating_ips, and networks since they are not tied to a user.<br>

<br>
The db migration issue is documented at <a href="https://bugs.launchpad.net/nova/+bug/1212798" target="_blank">https://bugs.launchpad.net/<u></u>nova/+bug/1212798</a> but the tl;dr is that the quota usages that were in place before the migration is run can not be decremented and aren't fixed by the healing sync that occurs.  I sought to address this by introducing a new migration which performs a full sync of quota usages and removes the bad rows but that led me to the next issue.<br>

<br>
Some resources can't be synced properly because they're tracked per user in the quota table but they're not tied to a user so it's not feasible to grab a count of how many are being used by any particular user.  So right now the quota_usages table can get into a bad state with no good way to address it.<br>

<br>
Right now I think it will be better to revert this change and re-introduce it once these issues are worked out. Thoughts?<br>
<br>
As an addendum, the patch merged about a month ago on Jul 25th and looks to have some minor conflicts for a revert but should be minimally disruptive.<br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</blockquote></div><br></div>