Mike Kolesnik has submitted this change and it was merged. Change subject: core: Fixes in SnapshotsManager ......................................................................
core: Fixes in SnapshotsManager 1. No need for new transaction when adding snapshot, just add it to the compensation context. 2. Added code to delete the disks that are not part of the snapshot restored to. 3. Configuration will save needed info by itself (no need to supply disks/nics). 4. Restoration will keep values that aren't saved in OVF. Change-Id: Ib6819f6e6f496e9ef01a234d3075a1a302eb7ba2 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java 1 file changed, 69 insertions(+), 17 deletions(-) Approvals: Mike Kolesnik: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2312 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6819f6e6f496e9ef01a234d3075a1a302eb7ba2 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Muli Salem <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
