gnt-cluster verify-disks: fix VG name
authorIustin Pop <iustin@google.com>
Wed, 26 Jan 2011 17:51:19 +0000 (18:51 +0100)
committerIustin Pop <iustin@google.com>
Wed, 26 Jan 2011 18:07:08 +0000 (19:07 +0100)
Recent multi-VG work already exports the missing LV names as vg/lv,
not simply lv. So the query and addition of the VG name in gnt-cluster
verify-disks is redundant, and even wrong for non-default-VG
instances.

Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com>

lib/client/gnt_cluster.py

index d430d48..21b79bf 100644 (file)
@@ -490,8 +490,6 @@ def VerifyDisks(opts, args):
         ToStderr("Error activating disks for instance %s: %s", iname, msg)
 
   if missing:
-    (vg_name, ) = cl.QueryConfigValues(["volume_group_name"])
-
     for iname, ival in missing.iteritems():
       all_missing = compat.all(x[0] in bad_nodes for x in ival)
       if all_missing:
@@ -502,9 +500,9 @@ def VerifyDisks(opts, args):
         ival.sort()
         for node, vol in ival:
           if node in bad_nodes:
-            ToStdout("\tbroken node %s /dev/%s/%s", node, vg_name, vol)
+            ToStdout("\tbroken node %s /dev/%s", node, vol)
           else:
-            ToStdout("\t%s /dev/%s/%s", node, vg_name, vol)
+            ToStdout("\t%s /dev/%s", node, vol)
 
     ToStdout("You need to run replace_disks for all the above"
              " instances, if this message persist after fixing nodes.")