Change in ovirt-engine[master]: core: StorageJobs - adding support for job fencing

Code Review gerrit at ovirt.org
Mon Jan 16 17:32:59 UTC 2017


>From Liron Aravot <laravot at redhat.com>:

Liron Aravot has submitted this change and it was merged.

Change subject: core: StorageJobs - adding support for job fencing
......................................................................


core: StorageJobs - adding support for job fencing

This patch adds support for storage job fencing for entity polling
commands in the StorageJobCallback.

When a job possibly hasn't started yet, we can attempt to fence the job
using the entity that may be used to poll the job  status. Fencing the job
means that the host that recieved it will fail when attempting to
execute it.
Fencing the jobb would usually be performed by executing an asynchronous
fencing command on the entity the job is supposed to be performed on.
Once the job was fenced, the entity poller in use should be able to determine
the job status as failed.

Change-Id: Iaf318884f0d9667d80eaff7f31bf4fdd39a6afb1
Bug-Url: http://bugzilla.redhat.com/1409125
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StorageJobCallback.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/EntityPollingCommand.java
2 files changed, 23 insertions(+), 1 deletion(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Freddy Rolland: Looks good to me, but someone else must approve
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf318884f0d9667d80eaff7f31bf4fdd39a6afb1
Gerrit-PatchSet: 16
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot 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: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list