Change in ovirt-engine[master]: gluster: consider rebalance failed case in stop rebalance

[ofrenkel at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server