Arik Hadas has uploaded a new change for review. Change subject: restapi: save/restore memory in snapshot by default ......................................................................
restapi: save/restore memory in snapshot by default Change-Id: If1558baec2f343c1cbc626aeacef4383e8dededc Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotsResource.java 2 files changed, 4 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/26/25626/1 diff --git a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java index 66662ad..e4bc698 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java +++ b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java @@ -51,8 +51,8 @@ public Response restore(Action action) { action.setAsync(false); TryBackToAllSnapshotsOfVmParameters tryBackParams = new TryBackToAllSnapshotsOfVmParameters(parentId, guid); - if (action.isSetRestoreMemory()) { - tryBackParams.setRestoreMemory(action.isRestoreMemory()); + if (!action.isSetRestoreMemory() || action.isRestoreMemory()) { + tryBackParams.setRestoreMemory(true); } if (action.isSetDisks()) { tryBackParams.setDisks(collection.mapDisks(action.getDisks())); diff --git a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotsResource.java b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotsResource.java index db2f10d..2bf6381 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotsResource.java +++ b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotsResource.java @@ -51,8 +51,8 @@ validateParameters(snapshot, "description"); CreateAllSnapshotsFromVmParameters snapshotParams = new CreateAllSnapshotsFromVmParameters(parentId, snapshot.getDescription()); - if (snapshot.isSetPersistMemorystate()) { - snapshotParams.setSaveMemory(snapshot.isPersistMemorystate()); + if (!snapshot.isSetPersistMemorystate() || snapshot.isPersistMemorystate()) { + snapshotParams.setSaveMemory(true); } if (snapshot.isSetDisks()) { snapshotParams.setDisks(mapDisks(snapshot.getDisks())); -- To view, visit http://gerrit.ovirt.org/25626 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: If1558baec2f343c1cbc626aeacef4383e8dededc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
