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

Fix backend.VerifyNode behaviour for VG problems



In case LVM is broken, backend.GetVolumeList will raise an RPC exception
(as expected since it's a function exposed over RPC). Therefore we must
be prepared to catch any such exceptions, so that we don't fail the
whole verify call in this case. cmdlib is already prepared to handle
string results for this response key.
Signed-off-by: default avatarIustin Pop <iustin@google.com>
Reviewed-by: default avatarGuido Trotter <ultrotter@google.com>
parent 2263aec2
......@@ -524,7 +524,11 @@ def VerifyNode(what, cluster_name):
" and ".join(fail))
if constants.NV_LVLIST in what:
result[constants.NV_LVLIST] = GetVolumeList(what[constants.NV_LVLIST])
try:
val = GetVolumeList(what[constants.NV_LVLIST])
except RPCFail, err:
val = str(err)
result[constants.NV_LVLIST] = val
if constants.NV_INSTANCELIST in what:
# GetInstanceList can fail
......
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