Commit 6973587f authored by Michael Hanselmann's avatar Michael Hanselmann
Browse files

OpGroupVerifyDisks: Fix wrong result type declaration



If an instance had actually a missing disk, the type check would fail.
Signed-off-by: default avatarMichael Hanselmann <hansmi@google.com>
Reviewed-by: default avatarIustin Pop <iustin@google.com>
parent 0b58db81
......@@ -3167,7 +3167,7 @@ class LUGroupVerifyDisks(NoHooksLU):
# any leftover items in nv_dict are missing LVs, let's arrange the data
# better
for key, inst in nv_dict.iteritems():
res_missing.setdefault(inst, []).append(key)
res_missing.setdefault(inst, []).append(list(key))
return (res_nodes, list(res_instances), res_missing)
......
......@@ -673,7 +673,8 @@ class OpGroupVerifyDisks(OpCode):
ht.TAnd(ht.TIsLength(3),
ht.TItems([ht.TDictOf(ht.TString, ht.TString),
ht.TListOf(ht.TString),
ht.TDictOf(ht.TString, ht.TListOf(ht.TString))]))
ht.TDictOf(ht.TString,
ht.TListOf(ht.TListOf(ht.TString)))]))
class OpClusterRepairDiskSizes(OpCode):
......
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