Change in ovirt-engine[master]: core: lock active snapshot during undo preview process

mlipchuk at redhat.com mlipchuk at redhat.com
Mon Jan 14 15:14:03 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: liron aravot <liron.aravot at gmail.com>



More information about the Engine-commits mailing list