From 6869f673b058cc58cf4b7a023f0220d0d047de55 Mon Sep 17 00:00:00 2001 From: Thomas Thrainer Date: Mon, 19 Aug 2013 15:51:23 +0200 Subject: [PATCH] Use node UUIDs for locking instead of node names LUInstanceMultiAlloc used node names instead of node UUIDs to lock required nodes. This patch fixes this bug. Signed-off-by: Thomas Thrainer Reviewed-by: Guido Trotter --- lib/cmdlib/instance.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/cmdlib/instance.py b/lib/cmdlib/instance.py index ac27f35..653ae60 100644 --- a/lib/cmdlib/instance.py +++ b/lib/cmdlib/instance.py @@ -1896,11 +1896,11 @@ class LUInstanceMultiAlloc(NoHooksLU): for inst in self.op.instances: (inst.pnode_uuid, inst.pnode) = \ ExpandNodeUuidAndName(self.cfg, inst.pnode_uuid, inst.pnode) - nodeslist.append(inst.pnode) + nodeslist.append(inst.pnode_uuid) if inst.snode is not None: (inst.snode_uuid, inst.snode) = \ ExpandNodeUuidAndName(self.cfg, inst.snode_uuid, inst.snode) - nodeslist.append(inst.snode) + nodeslist.append(inst.snode_uuid) self.needed_locks[locking.LEVEL_NODE] = nodeslist # Lock resources of instance's primary and secondary nodes (copy to -- 1.7.10.4