Change in ovirt-engine[master]: core: Compensation of active snapshot ID change

ofrenkel at redhat.com ofrenkel at redhat.com
Wed Aug 19 14:41:27 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: core: Compensation of active snapshot ID change
......................................................................


core: Compensation of active snapshot ID change

If exception is thrown inside CreateAllSnapshotsFromVmCommand, we need
to compensate the change of active snapshot ID that is performed during
snapshot creation. Otherwise, link between disk images and the active
snapshot is lost.

Change-Id: I432859105d31f74f10711f6a63b188d325a43e30
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1236061
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
1 file changed, 20 insertions(+), 3 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Verified
  Omer Frenkel: Looks good to me, approved
  Liron Aravot: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I432859105d31f74f10711f6a63b188d325a43e30
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
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: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Shmuel Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list