Commit f39b695a authored by Iustin Pop's avatar Iustin Pop
Browse files

Fix group verification of offline nodes

Commit aef59ae7

 reworked the file verification, but forgot to take
into account offline nodes.

The fact that this was not detected yet is due to the fact that we
don't test clusters with offline nodes in QA :(
Signed-off-by: default avatarIustin Pop <iustin@google.com>
Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
parent 6ba2bdc2
......@@ -1988,7 +1988,7 @@ class LUClusterVerifyGroup(LogicalUnit, _VerifyErrors):
@param all_nvinfo: RPC results
"""
node_names = frozenset(node.name for node in nodeinfo)
node_names = frozenset(node.name for node in nodeinfo if not node.offline)
assert master_node in node_names
assert (len(files_all | files_all_opt | files_mc | files_vm) ==
......@@ -2007,6 +2007,9 @@ class LUClusterVerifyGroup(LogicalUnit, _VerifyErrors):
fileinfo = dict((filename, {}) for filename in file2nodefn.keys())
for node in nodeinfo:
if node.offline:
continue
nresult = all_nvinfo[node.name]
if nresult.fail_msg or not nresult.payload:
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment