From 97dada8d604def2ac836465c6548a2c1260e87f5 Mon Sep 17 00:00:00 2001 From: Mark Washenberger Date: Tue, 29 Nov 2011 11:18:15 -0500 Subject: Use uuids for compute manager agent update. Related to blueprint internal-uuids. Change-Id: Ib4406f7fd4e5d91852d63785f596014cc3979fd2 --- nova/compute/manager.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'nova/compute') diff --git a/nova/compute/manager.py b/nova/compute/manager.py index 2eed738c1..db1eee6bf 100644 --- a/nova/compute/manager.py +++ b/nova/compute/manager.py @@ -921,17 +921,16 @@ class ComputeManager(manager.SchedulerDependentManager): self.driver.inject_file(instance_ref, path, file_contents) @exception.wrap_exception(notifier=notifier, publisher_id=publisher_id()) - @checks_instance_lock - def agent_update(self, context, instance_id, url, md5hash): + @checks_instance_lock_uuid + def agent_update(self, context, instance_uuid, url, md5hash): """Update agent running on an instance on this host.""" context = context.elevated() - instance_ref = self.db.instance_get(context, instance_id) - instance_id = instance_ref['id'] + instance_ref = self.db.instance_get_by_uuid(context, instance_uuid) instance_state = instance_ref['power_state'] expected_state = power_state.RUNNING if instance_state != expected_state: LOG.warn(_('trying to update agent on a non-running ' - 'instance: %(instance_id)s (state: %(instance_state)s ' + 'instance: %(instance_uuid)s (state: %(instance_state)s ' 'expected: %(expected_state)s)') % locals()) nm = instance_ref['name'] msg = _('instance %(nm)s: updating agent to %(url)s') % locals() -- cgit