Updated Branches:
  refs/heads/api_refactoring f4a90a151 -> f548099d0

api: annotate instance group Cmds

Signed-off-by: Prasanna Santhanam <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/f548099d
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/f548099d
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/f548099d

Branch: refs/heads/api_refactoring
Commit: f548099d08b7e0645d36193257c61b609e39b32d
Parents: 71522b8
Author: Prasanna Santhanam <[email protected]>
Authored: Fri Dec 21 16:12:10 2012 -0800
Committer: Prasanna Santhanam <[email protected]>
Committed: Fri Dec 21 16:12:10 2012 -0800

----------------------------------------------------------------------
 .../api/command/user/vmgroup/CreateVMGroupCmd.java |   11 ++++++-----
 .../api/command/user/vmgroup/DeleteVMGroupCmd.java |    4 ++--
 .../api/command/user/vmgroup/ListVMGroupsCmd.java  |    5 ++---
 .../api/command/user/vmgroup/UpdateVMGroupCmd.java |    4 ++--
 .../api/response/ProjectAccountResponse.java       |    3 +++
 5 files changed, 15 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/f548099d/api/src/org/apache/cloudstack/api/command/user/vmgroup/CreateVMGroupCmd.java
----------------------------------------------------------------------
diff --git 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/CreateVMGroupCmd.java 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/CreateVMGroupCmd.java
index ad2d5ee..bcc2644 100644
--- 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/CreateVMGroupCmd.java
+++ 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/CreateVMGroupCmd.java
@@ -43,15 +43,16 @@ public class CreateVMGroupCmd extends BaseCmd {
     @Parameter(name = ApiConstants.NAME, type = CommandType.STRING, required = 
true, description = "the name of the instance group")
     private String groupName;
 
-    @Parameter(name = ApiConstants.ACCOUNT, type = CommandType.STRING, 
description = "the account of the instance group. The account parameter must be 
used with the domainId parameter.")
+    @Parameter(name = ApiConstants.ACCOUNT, type = CommandType.STRING,
+            description = "the account of the instance group. The account 
parameter must be used with the domainId parameter.")
     private String accountName;
 
-    //@IdentityMapper(entityTableName = "domain")
-    @Parameter(name = ApiConstants.DOMAIN_ID, type = CommandType.LONG, 
description = "the domain ID of account owning the instance group", 
entityType=DomainResponse.class)
+    @Parameter(name = ApiConstants.DOMAIN_ID, type=CommandType.UUID, 
entityType=DomainResponse.class,
+            description = "the domain ID of account owning the instance group")
     private Long domainId;
 
-    //@IdentityMapper(entityTableName = "projects")
-    @Parameter(name = ApiConstants.PROJECT_ID, type = CommandType.LONG, 
description = "The project of the instance group", 
entityType=ProjectAccountResponse.class)
+    @Parameter(name = ApiConstants.PROJECT_ID, type=CommandType.UUID, 
entityType=ProjectAccountResponse.class,
+            description = "The project of the instance group")
     private Long projectId;
 
     // ///////////////////////////////////////////////////

http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/f548099d/api/src/org/apache/cloudstack/api/command/user/vmgroup/DeleteVMGroupCmd.java
----------------------------------------------------------------------
diff --git 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/DeleteVMGroupCmd.java 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/DeleteVMGroupCmd.java
index d81d5c4..5983b68 100644
--- 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/DeleteVMGroupCmd.java
+++ 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/DeleteVMGroupCmd.java
@@ -38,8 +38,8 @@ public class DeleteVMGroupCmd extends BaseCmd{
     //////////////// API parameters /////////////////////
     /////////////////////////////////////////////////////
 
-    //@IdentityMapper(entityTableName="instance_group")
-    @Parameter(name=ApiConstants.ID, type=CommandType.LONG, required=true, 
description="the ID of the instance group", 
entityType=InstanceGroupResponse.class)
+    @Parameter(name=ApiConstants.ID, type=CommandType.UUID, 
entityType=InstanceGroupResponse.class,
+            required=true, description="the ID of the instance group")
     private Long id;
 
     /////////////////////////////////////////////////////

http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/f548099d/api/src/org/apache/cloudstack/api/command/user/vmgroup/ListVMGroupsCmd.java
----------------------------------------------------------------------
diff --git 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/ListVMGroupsCmd.java 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/ListVMGroupsCmd.java
index 1aa51f8..a8906ed 100644
--- 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/ListVMGroupsCmd.java
+++ 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/ListVMGroupsCmd.java
@@ -20,7 +20,6 @@ import org.apache.log4j.Logger;
 
 import org.apache.cloudstack.api.ApiConstants;
 import org.apache.cloudstack.api.BaseListProjectAndAccountResourcesCmd;
-import org.apache.cloudstack.api.IdentityMapper;
 import org.apache.cloudstack.api.Implementation;
 import org.apache.cloudstack.api.Parameter;
 import org.apache.cloudstack.api.response.InstanceGroupResponse;
@@ -36,8 +35,8 @@ public class ListVMGroupsCmd extends 
BaseListProjectAndAccountResourcesCmd {
     //////////////// API parameters /////////////////////
     /////////////////////////////////////////////////////
 
-    @IdentityMapper(entityTableName="instance_group")
-    @Parameter(name=ApiConstants.ID, type=CommandType.LONG, description="list 
instance groups by ID", entityType=InstanceGroupResponse.class)
+    @Parameter(name=ApiConstants.ID, type=CommandType.UUID, 
entityType=InstanceGroupResponse.class,
+            description="list instance groups by ID")
     private Long id;
 
     @Parameter(name=ApiConstants.NAME, type=CommandType.STRING, 
description="list instance groups by name")

http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/f548099d/api/src/org/apache/cloudstack/api/command/user/vmgroup/UpdateVMGroupCmd.java
----------------------------------------------------------------------
diff --git 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/UpdateVMGroupCmd.java 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/UpdateVMGroupCmd.java
index 5762db9..3adf1f7 100644
--- 
a/api/src/org/apache/cloudstack/api/command/user/vmgroup/UpdateVMGroupCmd.java
+++ 
b/api/src/org/apache/cloudstack/api/command/user/vmgroup/UpdateVMGroupCmd.java
@@ -37,8 +37,8 @@ public class UpdateVMGroupCmd extends BaseCmd{
     //////////////// API parameters /////////////////////
     /////////////////////////////////////////////////////
 
-    //@IdentityMapper(entityTableName="instance_group")
-    @Parameter(name=ApiConstants.ID, type=CommandType.LONG, required=true, 
description="Instance group ID", entityType=InstanceGroupResponse.class)
+    @Parameter(name=ApiConstants.ID, type=CommandType.UUID, 
entityType=InstanceGroupResponse.class,
+            required=true, description="Instance group ID")
     private Long id;
 
     @Parameter(name=ApiConstants.NAME, type=CommandType.STRING, 
description="new instance group name")

http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/f548099d/api/src/org/apache/cloudstack/api/response/ProjectAccountResponse.java
----------------------------------------------------------------------
diff --git 
a/api/src/org/apache/cloudstack/api/response/ProjectAccountResponse.java 
b/api/src/org/apache/cloudstack/api/response/ProjectAccountResponse.java
index fadf423..da16291 100644
--- a/api/src/org/apache/cloudstack/api/response/ProjectAccountResponse.java
+++ b/api/src/org/apache/cloudstack/api/response/ProjectAccountResponse.java
@@ -18,12 +18,15 @@ package org.apache.cloudstack.api.response;
 
 import java.util.List;
 
+import com.cloud.projects.ProjectAccount;
 import org.apache.cloudstack.api.ApiConstants;
 import com.cloud.utils.IdentityProxy;
 import com.cloud.serializer.Param;
 import com.google.gson.annotations.SerializedName;
 import org.apache.cloudstack.api.BaseResponse;
+import org.apache.cloudstack.api.Entity;
 
+@Entity(value=ProjectAccount.class)
 @SuppressWarnings("unused")
 public class ProjectAccountResponse extends BaseResponse implements 
ControlledViewEntityResponse {
     @SerializedName(ApiConstants.PROJECT_ID)

Reply via email to