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

tnisan at redhat.com tnisan at redhat.com
Mon Aug 18 09:13:06 UTC 2014


Tal Nisan 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>
(cherry picked from commit c01ac6733886c0108937251ccbc4948309e41688)
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  Oved Ourfali: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9d00ea9d29acfa4894595a978a8376a530c2fc8d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list