Change in ovirt-engine[master]: core: Commiting a partial preview of cinder snapshot.

mlipchuk at redhat.com mlipchuk at redhat.com
Tue Dec 22 16:23:25 UTC 2015


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Commiting a partial preview of cinder snapshot.
......................................................................


core: Commiting a partial preview of cinder snapshot.

Critical snapshot list is list which reflects all the snapshot ids that are
being used for all the volumes in the VM.
Partial snapshot might contain a volume used for active snapshot.
Once the snapshot gets commited the engine should avoid removing the
snapshots which are being used by other volumes, indicated in the
critical snapshot list

Change-Id: I623ac84b1a0a869d25f0d8427ee5cd7631afda3e
Bug-Url: https://bugzilla.redhat.com/1293226
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RestoreAllSnapshotsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/connection/CINDERStorageHelper.java
2 files changed, 24 insertions(+), 12 deletions(-)

Approvals:
  Daniel Erez: Looks good to me, approved
  Maor Lipchuk: Verified; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I623ac84b1a0a869d25f0d8427ee5cd7631afda3e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list