Change in vdsm[master]: hsm: Check all devices visibility after multipath rescan

danken at redhat.com danken at redhat.com
Thu Dec 26 10:28:12 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hsm: Check all devices visibility after multipath rescan
......................................................................


hsm: Check all devices visibility after multipath rescan

After invoking multipath rescan, existing devices may disapper, and
non-existing devices may appear. The orignal code was checking again
only one device after a rescan, possibly returning bogus results.

This patch check now all devices after invoking multipath.rescan().

Change-Id: I46ede1124f4c5f849b040b598b49ee784c540ecc
Relates-To: https://bugzilla.redhat.com/923773
Bug-Url: https://bugzilla.redhat.com/1044068
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21182
Reviewed-by: Ayal Baron <abaron at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 5 insertions(+), 9 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Nir Soffer: Verified



-- 
To view, visit http://gerrit.ovirt.org/21182
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I46ede1124f4c5f849b040b598b49ee784c540ecc
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list