Pavel Zhukov has uploaded a new change for review.
Change subject: vgscan doesn't return nonzero return code if one or more phisycal volumes are filtered (missed). We should check vg before raising of the exception. chkVG itself raises storageAccessError if failed, so selftest doesn't have to it second time. ......................................................................
vgscan doesn't return nonzero return code if one or more phisycal volumes are filtered (missed). We should check vg before raising of the exception. chkVG itself raises storageAccessError if failed, so selftest doesn't have to it second time.
Change-Id: If1eeed1c203f2c8c73370987048565d665932299 Bugzilla-Url: https://bugzilla.redhat.com/1022976 Signed-off-by: Pavel Zhukov pzhukov@redhat.com --- M vdsm/storage/blockSD.py 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/52/20552/1
diff --git a/vdsm/storage/blockSD.py b/vdsm/storage/blockSD.py index b95be21..4b21c35 100644 --- a/vdsm/storage/blockSD.py +++ b/vdsm/storage/blockSD.py @@ -846,7 +846,7 @@ self._lastUncachedSelftest = now lvm.chkVG(self.sdUUID) elif lvm.getVG(self.sdUUID).partial != lvm.VG_OK: - raise se.StorageDomainAccessError(self.sdUUID) + lvm.chkVG(self.sdUUID)
def validate(self): """