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 <vvola...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/21089 Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Allon Mureinik <amure...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@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 <vvola...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Eduardo <ewars...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches