Node/Node group name is much easier for humans to read.

Signed-off-by: Brian Foley <[email protected]>
---
 lib/cmdlib/common.py | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/lib/cmdlib/common.py b/lib/cmdlib/common.py
index 1d79a3e..f902164 100644
--- a/lib/cmdlib/common.py
+++ b/lib/cmdlib/common.py
@@ -126,10 +126,11 @@ def CheckNodeGroupInstances(cfg, group_uuid, 
owned_instance_names):
   wanted_instances = frozenset(cfg.GetInstanceNames(
                                  cfg.GetNodeGroupInstances(group_uuid)))
   if owned_instance_names != wanted_instances:
+    group_name = cfg.GetNodeGroup(group_uuid).name
     raise errors.OpPrereqError("Instances in node group '%s' changed since"
                                " locks were acquired, wanted '%s', have '%s';"
                                " retry the operation" %
-                               (group_uuid,
+                               (group_name,
                                 utils.CommaJoin(wanted_instances),
                                 utils.CommaJoin(owned_instance_names)),
                                errors.ECODE_STATE)
@@ -1584,15 +1585,15 @@ def EnsureKvmdOnNodes(lu, feedback_fn, nodes=None):
   if start_nodes:
     results = lu.rpc.call_node_ensure_daemon(start_nodes, constants.KVMD, True)
     for node_uuid in start_nodes:
-      results[node_uuid].Warn("Failed to start KVM daemon in node '%s'" %
-                              node_uuid, feedback_fn)
+      results[node_uuid].Warn("Failed to start KVM daemon on node '%s'" %
+                              lu.cfg.GetNodeName(node_uuid), feedback_fn)
 
   # Stop KVM where necessary
   if stop_nodes:
     results = lu.rpc.call_node_ensure_daemon(stop_nodes, constants.KVMD, False)
     for node_uuid in stop_nodes:
-      results[node_uuid].Warn("Failed to stop KVM daemon in node '%s'" %
-                              node_uuid, feedback_fn)
+      results[node_uuid].Warn("Failed to stop KVM daemon on node '%s'" %
+                              lu.cfg.GetNodeName(node_uuid), feedback_fn)
 
 
 def WarnAboutFailedSshUpdates(result, master_uuid, feedback_fn):
-- 
2.8.0.rc3.226.g39d4020

Reply via email to