Maor Lipchuk has submitted this change and it was merged. Change subject: core: lock active snapshot during undo preview process ...................................................................... core: lock active snapshot during undo preview process This patch prevents scenarios in which commands that are dependent on the VM's snapshot are executing in parallel to undo preview command which is changing the active snapshot of the VM. The active snapshot of the VM is being locked when the async part of the RestoreAllSnapshots command begins, and will be changed to OK when the command ends. Change-Id: Id7a2064d5130a6fa49360a8ad8e9a1004fc44da2 Bug-Url: https://bugzilla.redhat.com/868648 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreAllSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java 2 files changed, 36 insertions(+), 2 deletions(-) Approvals: Maor Lipchuk: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/10845 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id7a2064d5130a6fa49360a8ad8e9a1004fc44da2 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: liron aravot <liron.aravot@gmail.com>