Updated Branches:
  refs/heads/master 42d33fc3f -> b130e8b3f

CLOUSTACK-997 improved documentation for assignVirtualMachine API; updated 
overall description and per parameter descriptions to per bug suggestions


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

Branch: refs/heads/master
Commit: b130e8b3f09a417dd278c59a55bfaaa5836937b4
Parents: 42d33fc
Author: David Grizzanti <david.grizza...@sungard.com>
Authored: Fri Mar 15 14:29:25 2013 -0400
Committer: Chip Childers <chip.child...@gmail.com>
Committed: Fri Mar 15 21:30:55 2013 -0400

----------------------------------------------------------------------
 .../api/command/admin/vm/AssignVMCmd.java          |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/b130e8b3/api/src/org/apache/cloudstack/api/command/admin/vm/AssignVMCmd.java
----------------------------------------------------------------------
diff --git 
a/api/src/org/apache/cloudstack/api/command/admin/vm/AssignVMCmd.java 
b/api/src/org/apache/cloudstack/api/command/admin/vm/AssignVMCmd.java
index 8a75c66..152dd4e 100644
--- a/api/src/org/apache/cloudstack/api/command/admin/vm/AssignVMCmd.java
+++ b/api/src/org/apache/cloudstack/api/command/admin/vm/AssignVMCmd.java
@@ -35,7 +35,7 @@ import org.apache.log4j.Logger;
 import com.cloud.user.Account;
 import com.cloud.uservm.UserVm;
 
-@APICommand(name = "assignVirtualMachine", description="Move a user VM to 
another user under same domain.", responseObject=UserVmResponse.class, 
since="3.0.0")
+@APICommand(name = "assignVirtualMachine", description="Assign a VM from one 
account to another under the same domain. This API is available for Basic zones 
with security groups and Advance zones with guest networks. The VM is 
restricted to move between accounts under same domain.", 
responseObject=UserVmResponse.class, since="3.0.0")
 public class AssignVMCmd extends BaseCmd  {
     public static final Logger s_logger = 
Logger.getLogger(AssignVMCmd.class.getName());
 
@@ -46,7 +46,7 @@ public class AssignVMCmd extends BaseCmd  {
     /////////////////////////////////////////////////////
 
     @Parameter(name=ApiConstants.VIRTUAL_MACHINE_ID, type=CommandType.UUID, 
entityType=UserVmResponse.class,
-            required=true, description="the vm ID of the user VM to be moved")
+            required=true, description="id of the VM to be moved")
     private Long virtualMachineId;
 
     @Parameter(name=ApiConstants.ACCOUNT, type=CommandType.STRING, 
required=true, description="account name of the new VM owner.")
@@ -58,11 +58,11 @@ public class AssignVMCmd extends BaseCmd  {
 
     //Network information
     @Parameter(name=ApiConstants.NETWORK_IDS, type=CommandType.LIST, 
collectionType=CommandType.UUID, entityType=NetworkResponse.class,
-            description="list of network ids that will be part of VM network 
after move in advanced network setting.")
+            description="list of new network ids in which the moved VM will 
participate. In case no network ids are provided the VM will be part of the 
default network for that zone. In case there is no network yet created for the 
new account the default network will be created.")
     private List<Long> networkIds;
 
     @Parameter(name=ApiConstants.SECURITY_GROUP_IDS, type=CommandType.LIST, 
collectionType=CommandType.UUID, entityType=SecurityGroupResponse.class,
-            description="comma separated list of security groups id that going 
to be applied to the virtual machine. Should be passed only when vm is moved in 
a zone with Basic Network support.")
+            description="list of security group ids to be applied on the 
virtual machine. In case no security groups are provided the VM is part of the 
default security group.")
     private List<Long> securityGroupIdList;
 
     /////////////////////////////////////////////////////

Reply via email to