-
Iustin Pop authored
Commit b8d26c6e added disk status verification, but it has two (different) bugs for not healthy nodes. For offline nodes, we don't add at all the disk status to the instance/node dict, with the result that the instance is not present in the instdisk dict if all of its nodes are offline. This creates a KeyError later when we call VerifyInstance with instdisk[instance]. For online nodes, but which don't return a valid disk status, we simply set the status to None for each disk, but the code in _VerifyInstance presumes and requires that each status is a valid tuple of length two. For both these bugs, we redo the instdisk computations to always include valid data, and we enhance the asserts to check for consistency. Signed-off-by: Iustin Pop <iustin@google.com> Reviewed-by: Guido Trotter <ultrotter@google.com>
d41d07d4