cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ahu...@apache.org
Subject [05/50] [abbrv] git commit: updated refs/heads/vmsync to 78fa5e1
Date Wed, 03 Jul 2013 22:25:44 GMT
CLOUDSTACK-3152 addVmwareDc API call fails if called more than once to add same VMware DC to
same cloudstack zone.

Check if the associated VMware DC matches the one specified in API params. This check would
yield success as the association exists between same entities (zone and VMware DC)
This scenario would result in if the API addVmwareDc is called more than once with same parameters.
For such scenario, return success response with VmwareDatacenter object read from database.
Also returns success if addVmwareDC is called agsin with same zone and VMwareDC where the
zone already has resources added to it (clusters etc.)

Signed-off-by: Sateesh Chodapuneedi <sateesh@apache.org>


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

Branch: refs/heads/vmsync
Commit: 6aa9b2f35657612e4ef9444971f556d152da5f9a
Parents: 02f207f
Author: Sateesh Chodapuneedi <sateesh@apache.org>
Authored: Tue Jul 2 17:31:09 2013 +0530
Committer: Sateesh Chodapuneedi <sateesh@apache.org>
Committed: Tue Jul 2 17:31:09 2013 +0530

----------------------------------------------------------------------
 .../vmware/manager/VmwareManagerImpl.java       | 48 +++++++++++++++-----
 1 file changed, 37 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/6aa9b2f3/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
----------------------------------------------------------------------
diff --git a/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
b/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
index fd99561..7b8697d 100755
--- a/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
+++ b/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
@@ -915,15 +915,6 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager,
Vmw
         String vCenterHost = cmd.getVcenter();
         String vmwareDcName = cmd.getName();
 
-        // Zone validation
-        validateZone(zoneId, "add VMware datacenter to zone");
-
-        VmwareDatacenterZoneMapVO vmwareDcZoneMap = _vmwareDcZoneMapDao.findByZoneId(zoneId);
-        // Check if zone is associated with VMware DC
-        if (vmwareDcZoneMap != null) {
-            throw new CloudRuntimeException("Zone " + zoneId + " is already associated with
a VMware datacenter.");
-        }
-
         // Validate username, password, VMware DC name and vCenter
         if (userName == null) {
             throw new InvalidParameterValueException("Missing or invalid parameter username.");
@@ -942,6 +933,36 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager,
Vmw
                     "Please provide valid VMware vCenter server's IP address or fully qualified
domain name.");
         }
 
+        if (zoneId == null) {
+            throw new InvalidParameterValueException("Missing or invalid parameter name.
" +
+                    "Please provide valid zone id.");
+        }
+
+        // Zone validation
+        validateZone(zoneId);
+
+        VmwareDatacenterZoneMapVO vmwareDcZoneMap = _vmwareDcZoneMapDao.findByZoneId(zoneId);
+        // Check if zone is associated with VMware DC
+        if (vmwareDcZoneMap != null) {
+            // Check if the associated VMware DC matches the one specified in API params
+            // This check would yield success as the association exists between same entities
(zone and VMware DC)
+            // This scenario would result in if the API addVmwareDc is called more than once
with same parameters.
+            Long associatedVmwareDcId = vmwareDcZoneMap.getVmwareDcId();
+            VmwareDatacenterVO associatedVmwareDc = _vmwareDcDao.findById(associatedVmwareDcId);
+            if (associatedVmwareDc.getVcenterHost().equalsIgnoreCase(vCenterHost) &&
+                    associatedVmwareDc.getVmwareDatacenterName().equalsIgnoreCase(vmwareDcName))
{
+                s_logger.info("Ignoring API call addVmwareDc, because VMware DC " + vCenterHost
+ "/" + vmwareDcName +
+                              " is already associated with specified zone with id " + zoneId);
+                return associatedVmwareDc;
+            } else {
+                throw new CloudRuntimeException("Zone " + zoneId + " is already associated
with a VMware datacenter. " +
+                                                "Only 1 VMware DC can be associated with
a zone.");
+            }
+        }
+        // Zone validation to check if the zone already has resources.
+        // Association of VMware DC to zone is not allowed if zone already has resources
added.
+        validateZoneWithResources(zoneId, "add VMware datacenter to zone");
+
         // Check if DC is already part of zone
         // In that case vmware_data_center table should have the DC
         vmwareDc = _vmwareDcDao.getVmwareDatacenterByGuid(vmwareDcName + "@" + vCenterHost);
@@ -1037,7 +1058,10 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager,
Vmw
     public boolean removeVmwareDatacenter(RemoveVmwareDcCmd cmd) throws ResourceInUseException
{
         Long zoneId = cmd.getZoneId();
         // Validate zone
-        validateZone(zoneId, "remove VMware datacenter from zone");
+        validateZone(zoneId);
+        // Zone validation to check if the zone already has resources.
+        // Association of VMware DC to zone is not allowed if zone already has resources
added.
+        validateZoneWithResources(zoneId, "remove VMware datacenter to zone");
 
         // Get DC associated with this zone
         VmwareDatacenterZoneMapVO vmwareDcZoneMap;
@@ -1107,7 +1131,7 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager,
Vmw
         return true;
     }
 
-    private void validateZone(Long zoneId, String errStr) throws ResourceInUseException {
+    private void validateZone(Long zoneId) throws ResourceInUseException {
         // Check if zone with specified id exists
         DataCenterVO zone = _dcDao.findById(zoneId);
         if (zone == null) {
@@ -1115,7 +1139,9 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager,
Vmw
                     "Can't find zone by the id specified.");
             throw ex;
         }
+    }
 
+    private void validateZoneWithResources(Long zoneId, String errStr) throws ResourceInUseException
{
         // Check if zone has resources? - For now look for clusters
         List<ClusterVO> clusters = _clusterDao.listByZoneId(zoneId);
         if (clusters != null && clusters.size() > 0) {


Mime
View raw message