From 334d7b8d25b0aa9b41b578df8309497e34b0a05f Mon Sep 17 00:00:00 2001 From: Brian Elliott Date: Wed, 2 Jan 2013 19:24:00 +0000 Subject: Fix race in resource tracker Fix a race in resource tracker where usage can get under reported if the audit process runs after an instance gets set to vm_state=DELETED, but before the DB record itself is deleted. bug 1095403 Change-Id: I47943d12155a6484c6cb3adb31dfa4db0191cf26 --- nova/compute/resource_tracker.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'nova/compute') diff --git a/nova/compute/resource_tracker.py b/nova/compute/resource_tracker.py index 1c653b8ca..c784fd83d 100644 --- a/nova/compute/resource_tracker.py +++ b/nova/compute/resource_tracker.py @@ -503,7 +503,10 @@ class ResourceTracker(object): resources['running_vms'] = 0 for instance in instances: - self._update_usage_from_instance(resources, instance) + if instance['vm_state'] == vm_states.DELETED: + continue + else: + self._update_usage_from_instance(resources, instance) def _find_orphaned_instances(self): """Given the set of instances and migrations already account for -- cgit