Fix misleading error message when checking disks
authorGuido Trotter <ultrotter@google.com>
Wed, 2 Apr 2008 18:06:04 +0000 (18:06 +0000)
committerGuido Trotter <ultrotter@google.com>
Wed, 2 Apr 2008 18:06:04 +0000 (18:06 +0000)
_CheckDiskConsistency outputs "Can't get any data from node NODE" when no drbd
is found on the target node. This causes a misleading error message to be
output for example on failover (when the primary node is down, or the instance
is not running), stating that no data could be got from the secondary node,
which scares the user and misleads him. Changing this to "Disk degraded or not
found on node %s" still reports that something is missing, but on the other
hand doesn't make the user think the node is down, or has no data at all...

Reviewed-by: imsnah

lib/cmdlib.py

index 96e028d..e00d4e2 100644 (file)
@@ -1154,7 +1154,7 @@ def _CheckDiskConsistency(cfgw, dev, node, on_primary, ldisk=False):
   if on_primary or dev.AssembleOnSecondary():
     rstats = rpc.call_blockdev_find(node, dev)
     if not rstats:
-      logger.ToStderr("Can't get any data from node %s" % node)
+      logger.ToStderr("Disk degraded or not found on node %s" % node)
       result = False
     else:
       result = result and (not rstats[idx])