Change in ovirt-engine[master]: core: lock VM for whole live snapshot process

ofrenkel at redhat.com ofrenkel at redhat.com
Wed Sep 25 08:41:16 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: lock VM for whole live snapshot process
......................................................................


core: lock VM for whole live snapshot process

VM was used to be locked for the entire live snapshot process only in
case the snapshot contained memory. If the snapshot did not contain
memory, the lock was released in the execute state of the command.

This patch changes that, such that the VM will be locked for the entire
live snapshot process in any case. By keeping the VM locked for the
entire process, we prevent other operations which should not run in
parallel to run, as long as they also try to lock the VM.

Change-Id: Ib455a3b27d7b032c6a706753a6e58e07d3a1bc14
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
1 file changed, 0 insertions(+), 7 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib455a3b27d7b032c6a706753a6e58e07d3a1bc14
Gerrit-PatchSet: 5
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: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list