Change in ovirt-engine[master]: core: Fix errors for retry of failed DestroyImageCommand

amureini at redhat.com amureini at redhat.com
Sun May 3 09:12:39 UTC 2015


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Fix errors for retry of failed DestroyImageCommand
......................................................................


core: Fix errors for retry of failed DestroyImageCommand

Fix issues encountered while re-executing Live Merge after a failure of
DestroyImageCommand:

1. Some images may have been marked illegal from a prior failure; this
is an expected condition so allow live merge execution in this case.

2. The endSuccessfully and endWithFailure methods should always indicate
their own success using setSucceeded(true) to avoid endAction retries.

3. If the DestroyImage SPM async task was started and converged before
CoCo called doPolling() again on RemoveSnapshotSingleDiskLive, the child
command list was not updated.  Subsequently the end action methods were
not able to call endAction on DestroyImageCommand and no commands in the
Live Merge command tree would converge.  This is fixed by updating the
child command list before referencing it to ensure all children are
present.

Change-Id: Icbbcc30fa035de82de05c5c60ce224ed36aee831
Related-To: https://bugzilla.redhat.com/1213157
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskLiveCommand.java
2 files changed, 38 insertions(+), 21 deletions(-)

Approvals:
  Greg Padgett: Verified
  Allon Mureinik: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icbbcc30fa035de82de05c5c60ce224ed36aee831
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list