Comment 8 for bug 1296414

Revision history for this message
jichenjc (jichenjc) wrote :

in _reclaim_queued_deletes there is
        # FIXME(comstud): Dummy quota object for now. See bug 1296414.
        # We have potential for inconsistency if we decide here to not
        # update quotas. _delete_instance() should determine whether or
        # not to update quotas based on if instance is in a SOFT_DELETED
        # state or not.

I think it's reasonable
soft_delete_instance will update the instance to soft_delete first then commit the quota
_init_instance will not handle SOFT_DELETE instance and leave it to _reclaim_queued_deletes

in this case, we need to commit the quota , I will submit a patch for it

let me know your comments, thanks