You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ts...@apache.org on 2013/04/18 04:26:25 UTC

[37/41] git commit: updated refs/heads/marvin_refactor to 94b18c6

CLOUDSTACK-2069: Anti-Affinity - listVirtualmachines - affinitygroup entity does not include other Vms that are part of this affinity group.

Removed the virtualmachineIds:[] sent in the API response.
The listVirtualMachines Resopnse need not list details beyond the id, name,description etc.
If the user wants to get the list of VMs in the group, listAffinityGroups APi can be used.


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

Branch: refs/heads/marvin_refactor
Commit: 3a9c63d8c919deaaea12d40f83272af3162e84d6
Parents: 6dccf63
Author: Prachi Damle <pr...@cloud.com>
Authored: Wed Apr 17 17:48:52 2013 -0700
Committer: Prachi Damle <pr...@cloud.com>
Committed: Wed Apr 17 18:04:15 2013 -0700

----------------------------------------------------------------------
 .../cloudstack/affinity/AffinityGroupResponse.java |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/3a9c63d8/api/src/org/apache/cloudstack/affinity/AffinityGroupResponse.java
----------------------------------------------------------------------
diff --git a/api/src/org/apache/cloudstack/affinity/AffinityGroupResponse.java b/api/src/org/apache/cloudstack/affinity/AffinityGroupResponse.java
index afd33da..b6d4ff6 100644
--- a/api/src/org/apache/cloudstack/affinity/AffinityGroupResponse.java
+++ b/api/src/org/apache/cloudstack/affinity/AffinityGroupResponse.java
@@ -64,7 +64,6 @@ public class AffinityGroupResponse extends BaseResponse implements ControlledVie
     private List<String> vmIdList;
 
     public AffinityGroupResponse() {
-        this.vmIdList = new ArrayList<String>();
     }
 
     @Override
@@ -149,6 +148,10 @@ public class AffinityGroupResponse extends BaseResponse implements ControlledVie
     }
 
     public void addVMId(String vmId) {
+        if (this.vmIdList == null) {
+            this.vmIdList = new ArrayList<String>();
+        }
+
         this.vmIdList.add(vmId);
     }