Change in ovirt-engine[master]: core: split ExecutionHandler.endTaskJob()

laravot at redhat.com laravot at redhat.com
Tue Apr 5 12:51:18 UTC 2016


Liron Aravot has submitted this change and it was merged.

Change subject: core: split ExecutionHandler.endTaskJob()
......................................................................


core: split ExecutionHandler.endTaskJob()

ExecutionHandler.endTaskJob() performs two different things today -
1. Ends the FINALIZING step if present, otherwise the current step.
2. Ends the jobs if needed

This patch splits this method into two different methods to be used
separately in the following patches.

Change-Id: I75f97b4347dce4b7e5130a7ae59e29013d24f973
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1310426
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterAsyncCommandBase.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/tasks/GlusterTaskUtils.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/job/ExecutionHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/lsm/LiveMigrateDisksTaskHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandAsyncTask.java
5 files changed, 65 insertions(+), 31 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Looks good to me, but someone else must approve
  Freddy Rolland: Looks good to me, but someone else must approve
  Moti Asayag: Looks good to me, approved
  Liron Aravot: Verified



-- 
To view, visit https://gerrit.ovirt.org/55613
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I75f97b4347dce4b7e5130a7ae59e29013d24f973
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list