Omer Frenkel has submitted this change and it was merged. Change subject: gluster: consider rebalance failed case in stop rebalance ...................................................................... gluster: consider rebalance failed case in stop rebalance Considering the rebalance failed case in stop rebalance BLL and update the task status correctly. Mark the task status same as the status returned from stop rebalance vdsm. If status is empty, mark the status as aborted. Refactored the GlusterTaskUtils.isTaskSuccess() to accept JobExecutionStatus instead of step. Change-Id: I147c43c6925b8a52cf8a7667a5d7f101969f5368 Bug-Url: https://bugzilla.redhat.com/1019598 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopRebalanceGlusterVolumeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/tasks/GlusterTaskUtils.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/StopRebalanceGlusterVolumeCommandTest.java 3 files changed, 13 insertions(+), 10 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21339 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I147c43c6925b8a52cf8a7667a5d7f101969f5368 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu@redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu@redhat.com> Gerrit-Reviewer: Sahina Bose <sabose@redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server