Michael Pasternak has posted comments on this change. Change subject: restapi : undo snapshot preview through restapi commits snapshot(#1018554) ......................................................................
Patch Set 1: Code-Review-2 (1 comment) it was decided to revert this RFE [1] as it doesn't have proper backend impl, that works for restapi. .................................................... File backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java Line 100: Response response = Response.status(Response.Status.BAD_REQUEST).entity(f).build(); Line 101: throw new WebApplicationException(response); Line 102: } Line 103: RestoreAllSnapshotsParameters restoreParams = new RestoreAllSnapshotsParameters(parentId, guid); Line 104: Response response = doAction(VdcActionType.RestoreAllSnapshots, did you noticed that both commit & undo uses VdcActionType.RestoreAllSnapshots? ........ Line 105: restoreParams, Line 106: action); Line 107: return response; Line 108: } -- To view, visit http://gerrit.ovirt.org/20625 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Icb5b226497664ded1f3c2c99d979a6da9e7a75aa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <[email protected]> Gerrit-Reviewer: Michael Pasternak <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
