When a remote node returns invalid LVM data, we check it, but we don't
stop and continue with the rest of the checks (which require a valid
volume group). This raises an internal error and breaks verify disks.

This seems unchanged for a long while, I don't know why it surfaced just
recently.
---
 lib/cmdlib.py |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/lib/cmdlib.py b/lib/cmdlib.py
index b8fd0be..bf57c27 100644
--- a/lib/cmdlib.py
+++ b/lib/cmdlib.py
@@ -1288,10 +1288,11 @@ class LUVerifyDisks(NoHooksLU):
         continue
       lvs = lvs.data
       if isinstance(lvs, basestring):
         logging.warning("Error enumerating LVs on node %s: %s", node, lvs)
         res_nlvm[node] = lvs
+        continue
       elif not isinstance(lvs, dict):
         logging.warning("Connection to node %s failed or invalid data"
                         " returned", node)
         res_nodes.append(node)
         continue
-- 
1.6.2.4

Reply via email to