Change in vdsm[master]: hsm: Do not hide errors when checking device visibility

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


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hsm: Do not hide errors when checking device visibility
......................................................................


hsm: Do not hide errors when checking device visibility

When checking devices visibility, all errors from os.stat() are treated
as "device does not exists", hiding fatal error and making it harder to
debug and fix them.

This patch handle only ENOENT for checking visibility and raise other
errors we cannot handle.

Change-Id: I7411d7d2eaa9172f612fd1b1e04177bef270ab19
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/21211
Reviewed-by: Ayal Baron <abaron at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 3 insertions(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7411d7d2eaa9172f612fd1b1e04177bef270ab19
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