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(a)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
Jenkins CI: Passed CI tests
Oved Ourfali: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/47088
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I984f06d05f1ac43985cb9a8bda6dad7cfa5ed3ef
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org