Change in vdsm[master]: Improve error when multipath can't access a pv

danken at redhat.com danken at redhat.com
Sun Nov 17 08:43:29 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Improve error when multipath can't access a pv
......................................................................


Improve error when multipath can't access a pv

InvalidPhysDev gave a cryptic error message - Invalid physical device.
The message was amended to "Multipath cannot access physical device(s)" along
with the device(s) name(s). The exception name was changed according to the new
message content. Note: The only thing needed here by the oVirt engine is the
error code, which was not changed in this patch, so there are no
compatibility issues with older oVirt engines.

Change-Id: I648ee519873c51573e6e6306b79380f54bb25d2e
Bug-url: https://bugzilla.redhat.com/852003
Signed-off-by: Vered Volansky <vvolansk at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21089
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/storage/blockSD.py
M vdsm/storage/hsm.py
M vdsm/storage/storage_exception.py
3 files changed, 14 insertions(+), 12 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Allon Mureinik: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Vered Volansky: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I648ee519873c51573e6e6306b79380f54bb25d2e
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <vvolansk 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: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list