Change in ovirt-engine[master]: core: Revert Delete jobs that their steps have no async-tasks

oourfali at redhat.com oourfali at redhat.com
Tue Apr 1 13:37:04 UTC 2014


Oved Ourfali has submitted this change and it was merged.

Change subject: core: Revert Delete jobs that their steps have no async-tasks
......................................................................


core: Revert Delete jobs that their steps have no async-tasks

Patch 22474 and 26056 added an ability to remove jobs that have
no steps an no async-steps. This caused long running jobs to be
removed while running, causing insert errors and removing them
from customer use.

This patch removes that functionality. If jobs fail, they should
be marked as such. And if they are not, it is the job writer
responsibility to fix this behavior as a bug.

Change-Id: Ief49517c37188839572cd4e95a25b2f921f4b5e1
Bug-Url: https://bugzilla.redhat.com/1076246
Signed-off-by: lzelkha at redhat.com <lzelkha at redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/JobDaoTest.java
M packaging/dbscripts/job_sp.sql
2 files changed, 2 insertions(+), 13 deletions(-)

Approvals:
  Liran Zelkha: Verified
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/26291
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ief49517c37188839572cd4e95a25b2f921f4b5e1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list