Commit b5cbddd9 authored by René Nussbaumer's avatar René Nussbaumer
Browse files

Annotate disk params on blockdev_getmirrorstatus_multi



This is also related to the bug reported by Constantinos,
as we've only one getmirrorstatus_multi call in whole cmdlib, we just
annotate them while we are building the disk list.
Signed-off-by: default avatarRené Nussbaumer <rn@google.com>
Reviewed-by: default avatarAgata Murawska <agatamurawska@google.com>
parent 70817cee
......@@ -2924,12 +2924,12 @@ class LUClusterVerifyGroup(LogicalUnit, _VerifyErrors):
 
node_disks[nname] = disks
 
# Creating copies as SetDiskID below will modify the objects and that can
# lead to incorrect data returned from nodes
devonly = [dev.Copy() for (_, dev) in disks]
for dev in devonly:
self.cfg.SetDiskID(dev, nname)
# _AnnotateDiskParams makes already copies of the disks
devonly = []
for (inst, dev) in disks:
(anno_disk,) = _AnnotateDiskParams(instanceinfo[inst], [dev], self.cfg)
self.cfg.SetDiskID(anno_disk, nname)
devonly.append(anno_disk)
 
node_disks_devonly[nname] = devonly
 
......
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