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

tnisan at redhat.com tnisan at redhat.com
Wed Dec 23 12:50:05 UTC 2015


Tal Nisan 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/RestoreAllSnapshotsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CINDERStorageHelper.java
2 files changed, 24 insertions(+), 14 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I623ac84b1a0a869d25f0d8427ee5cd7631afda3e
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
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: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list