Kanagaraj M has uploaded a new change for review. Change subject: gluster: fix forwarding vdsm errors ......................................................................
gluster: fix forwarding vdsm errors set 'null' to VdcFault.Error to make the frontend to show the error returned by vdsm instead of "Internal Server Error". Change-Id: I5df008d6626da5ff0de4ad6231ef65f51e08ec84 Bug-Url: https://bugzilla.redhat.com/956159 Signed-off-by: Kanagaraj M <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/22/15022/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java index 7f78ba5..848fbb8 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java @@ -104,6 +104,7 @@ errorType = errType; getReturnValue().getExecuteFailedMessages().addAll(errors); getReturnValue().getFault().setMessage(StringUtils.join(errors, SystemUtils.LINE_SEPARATOR)); + getReturnValue().getFault().setError(null); } @@ -111,6 +112,7 @@ errorType = errType; getReturnValue().getExecuteFailedMessages().add(error); getReturnValue().getFault().setMessage(error); + getReturnValue().getFault().setError(null); } protected boolean updateBrickServerNames(List<GlusterBrickEntity> bricks, boolean addCanDoActionMessage) { -- To view, visit http://gerrit.ovirt.org/15022 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I5df008d6626da5ff0de4ad6231ef65f51e08ec84 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
