Change in ovirt-engine[ovirt-engine-3.6]: core: when starting Live Merge, check for NOT_STARTED child ...
tnisan at redhat.com
tnisan at redhat.com
Wed Nov 11 14:12:34 UTC 2015
Tal Nisan has submitted this change and it was merged.
Change subject: core: when starting Live Merge, check for NOT_STARTED child commands
......................................................................
core: when starting Live Merge, check for NOT_STARTED child commands
When a Live Merge is started, the RemoveSnapshotCommandCallback does not
look for child commands with the NOT_STARTED command status. This
causes a race where the child commands may not be detected properly by
the parent, causing the actual merge process to be skipped.
This adds the needed check to the command callback, eliminating the race
condition and ensuring the merge will occur.
Change-Id: If47d26d8bedaf0bb4a39cae472533399178b4fdd
Bug-Url: https://bugzilla.redhat.com/1280160
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommandCallback.java
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Tal Nisan: Verified; Passed CI tests
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/48427
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If47d26d8bedaf0bb4a39cae472533399178b4fdd
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the Engine-commits
mailing list