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