summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Prince <dan.prince@rackspace.com>2012-02-01 20:50:51 -0500
committerDan Prince <dan.prince@rackspace.com>2012-02-01 20:54:51 -0500
commit233f5a738f39bae9b815383cb0ba2f3c207ae813 (patch)
tree86158c0c426567476e092b681b73ab989dfa8eac
parent5db0142d2f96e8e2d152aea59e7fc53d1e5ca012 (diff)
Fix admin password skip check.
Skip admin password configuration only if the agent 'version' is empty. Previously both the 'version' and 'agent_build' were required to be set. As the 'agent_build' table is primarily used for upgrades there should be no requirement that a record exist in order to configure admin passwords. Fixes LP Bug #925237. Change-Id: Ie4530a30ed02f3d49a07037e9d1cc52ef0af1dd4
-rw-r--r--nova/virt/xenapi/vmops.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/nova/virt/xenapi/vmops.py b/nova/virt/xenapi/vmops.py
index f3ff9a22d..3e90ce9e4 100644
--- a/nova/virt/xenapi/vmops.py
+++ b/nova/virt/xenapi/vmops.py
@@ -501,7 +501,7 @@ class VMOps(object):
# if the guest agent is not available, configure the
# instance, but skip the admin password configuration
- no_agent = version is None or agent_build is None
+ no_agent = version is None
self._configure_instance(ctx, instance, vm_ref, no_agent)
def _handle_spawn_error(self, vdis, spawn_error):