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

smizrahi at redhat.com smizrahi at redhat.com
Tue May 29 14:15:56 UTC 2012


Saggi Mizrahi has posted comments on this change.

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


Patch Set 1: I would prefer that you didn't submit this

This is not the way to go. Either fix the problem in python and have the exception construct correctly on the client side or have super vdsm bypass the built in mechanism and pass exceptions in the response in the form of (res, err) and reraise them if err is not None.

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

Gerrit-MessageType: comment
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: Bala.FA <barumuga 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