Michael Pasternak has posted comments on this change.
Change subject: resapi: adding support for moving a VM disk
......................................................................
Patch Set 1: (2 inline comments)
....................................................
File
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml
Line 366: request:
Line 367: body:
Line 368: parameterType: Action
Line 369: signatures:
Line 370: - mandatoryArguments: {storagedomain.host.id|name: 'xs:string'}
correct (probably copy/paster leftover), you could figure out that according to
validateParameters(action, "storageDomain.id|name");
validation you did.
Line 371: optionalArguments: {action.async: 'xs:boolean'}
Line 372: urlparams: {}
Line 373: headers:
Line 374: Content-Type: {value: application/xml|json, required: true}
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDiskResource.java
Line 82: MoveDisksParameters params =
Line 83: new MoveDisksParameters(Collections.singletonList(new
MoveDiskParameters(
Line 84: asGuid(get().getImageId()),
Line 85: Guid.Empty,
Line 86: getStorageDomainId(action))));
both id/name should be supported, you can add GetStorageDomainByName or use
GetAllStorageDomains+loop
Line 87: return doAction(VdcActionType.MoveDisks, params, action);
Line 88: }
Line 89:
Line 90: @Override
--
To view, visit http://gerrit.ovirt.org/10676
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1c864a81463388f8ac82eba6f909c5a4359efab6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: Ori Liel <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
