Mike Kolesnik has submitted this change and it was merged. Change subject: core: Simplify RemoveSnapshot command to get id ......................................................................
core: Simplify RemoveSnapshot command to get id The RemoveSnapshot command will get the ID of the snapshot to remove, and will take care of figuring out who to merge to by itself. This is agreeed with REST/UI behaviour and also in accordance to planned changes on VDSM layer which is planned to do the merge of images internally. Change-Id: I7b79071550b4b53774ea9482a0ca2eda2f65b1fd --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RemoveSnapshotParameters.java 2 files changed, 14 insertions(+), 51 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1130 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b79071550b4b53774ea9482a0ca2eda2f65b1fd Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
