Change in ovirt-engine[master]: engine : CoCo Handle restart logic for commands

oourfali at redhat.com oourfali at redhat.com
Mon Aug 18 08:42:31 UTC 2014


Oved Ourfali has submitted this change and it was merged.

Change subject: engine : CoCo Handle restart logic for commands
......................................................................


engine : CoCo Handle restart logic for commands

When a command has failed due to an exception
in executeCommand method the status of the command
is set to FAILED and the execute flag is false.

If the server is restarted before the callback is
notified the status of the command should not be
set to FAILED_RESTARTED and the callback's onFailed
should called after restart for both FAILED
and FAILED_RESTARTED commands

Change-Id: I9d00ea9d29acfa4894595a978a8376a530c2fc8d
Bug-Url: https://bugzilla.redhat.com/1129885
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
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  Ravi Nori: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9d00ea9d29acfa4894595a978a8376a530c2fc8d
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori 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
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list