Change in vdsm[master]: Fix - bz #823565 - Gluster - Backend: Replace Brick presente...

barumuga at redhat.com barumuga at redhat.com
Thu May 24 09:29:17 UTC 2012


Hello Ayal Baron, Timothy Asir, Saggi Mizrahi, Federico Simoncelli, Dan Kenigsberg,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/4750

to review the following change.

Change subject: Fix - bz #823565 - Gluster - Backend: Replace Brick presented as failure
......................................................................

Fix - bz #823565 - Gluster - Backend: Replace Brick presented as failure

Gluster cli functions throw error as exception.  As these functions
are bound to supervdsm and exceptions are not properly raised by
supervdsm.  Unexpected exception is thrown by default.

This fix catches VdsmException at supervdsm.ProxyCaller and returns
error dictionary to the caller.

Change-Id: I08bc3d370c0ff503cf204580f4e10d11419cd48a
Signed-off-by: Bala.FA <barumuga at redhat.com>
---
M vdsm/gluster/api.py
M vdsm/supervdsm.py
2 files changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/50/4750/1
--
To view, visit http://gerrit.ovirt.org/4750
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I08bc3d370c0ff503cf204580f4e10d11419cd48a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Bala.FA <barumuga 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: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>


More information about the vdsm-patches mailing list