Change in ovirt-engine[master]: engine : CoCo infrastructure should provide a timeout for ea...

[oourfali at redhat.com]

Oved Ourfali has submitted this change and it was merged.

Change subject: engine : CoCo infrastructure should provide a timeout for each command
......................................................................


engine : CoCo infrastructure should provide a timeout for each command

A new vdc_option in the database CoCoCommandLifeInMinutes
determines the lifetime of a command. A command that is
still in Active status after the lifetime is marked as Failed.
The onFailed method is invoked on the command's callback which
is responsible for releasing any locks on the entities.

Commands can fine tune the life time of a command by
invoking VdcActionParametersBase.setLifeInMinutes

Change-Id: I984f06d05f1ac43985cb9a8bda6dad7cfa5ed3ef
Bug-Url: https://bugzilla.redhat.com/1261733
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionParametersBase.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/CommandEntity.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/CommandEntityDaoImpl.java
M packaging/dbscripts/command_entities_sp.sql
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
7 files changed, 55 insertions(+), 2 deletions(-)

Approvals:
  Ravi Nori: Verified
  Oved Ourfali: Looks good to me, approved; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I984f06d05f1ac43985cb9a8bda6dad7cfa5ed3ef
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: automation at ovirt.org