-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8133/#review13653
-----------------------------------------------------------
Hi Fang,
The entityType attribute on the @parameter should point to the actual interface
representing that entity in CloudStack. Not the Response Class for that entity.
For example:
@Parameter(name=ApiConstants.VIRTUAL_MACHINE_ID, type=CommandType.LONG,
required=true, description="the vm ID of the user VM to be moved",
entityType=VirtualMachine.class)
private Long virtualMachineId;
@Parameter(name=ApiConstants.DOMAIN_ID, type=CommandType.LONG,
required=true, description="domain id of the new VM owner.",
entityType=Domain.class)
private Long domainId;
- Prachi Damle
On Nov. 20, 2012, 2:05 a.m., Fang Wang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8133/
> -----------------------------------------------------------
>
> (Updated Nov. 20, 2012, 2:05 a.m.)
>
>
> Review request for cloudstack, Prachi Damle and Alex Huang.
>
>
> Description
> -------
>
> Part of the API refactoring work. This is the initial modification to the
> @Parameter annotation for
> some commands in the VM group.
>
>
> This addresses bug cloudstack-518.
>
>
> Diffs
> -----
>
> api/src/com/cloud/api/Parameter.java 2da3c40
> api/src/org/apache/cloudstack/user/api/vm/commands/AssignVMCmd.java
> PRE-CREATION
> api/src/org/apache/cloudstack/user/api/vm/commands/CreateVMGroupCmd.java
> PRE-CREATION
> api/src/org/apache/cloudstack/user/api/vm/commands/DeleteVMGroupCmd.java
> PRE-CREATION
> api/src/org/apache/cloudstack/user/api/vm/commands/DeployVMCmd.java
> PRE-CREATION
> api/src/org/apache/cloudstack/user/api/vm/commands/DestroySystemVmCmd.java
> PRE-CREATION
>
> Diff: https://reviews.apache.org/r/8133/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Fang Wang
>
>
