CSTACKEX-98 Checking Map value

Author: Gupta, Surya

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/lifecycle/OntapPrimaryDatastoreLifecycle.java

@@ -87,6 +87,8 @@ public DataStore initialize(Map<String, Object> dsInfos) {
             throw new CloudRuntimeException("Datastore info map is null, cannot create primary storage");
         }
         s_logger.info("initialize::::::::::::::: dsInfos " + dsInfos.toString());
+        Map<String, String> details = (Map<String, String>) dsInfos.get("details");
+        s_logger.info("initialize details::::::::::::::: details " + details.toString());
         String url = (String) dsInfos.get("url");
         Long zoneId = (Long) dsInfos.get("zoneId");
         Long podId = (Long) dsInfos.get("podId");
@@ -103,8 +105,7 @@ public DataStore initialize(Map<String, Object> dsInfos) {
         s_logger.debug("Received capacityBytes from UI: " + capacityBytes);
 
         // Additional details requested for ONTAP primary storage pool creation
-        @SuppressWarnings("unchecked")
-        Map<String, String> details = (Map<String, String>) dsInfos.get("details");
+        //Map<String, String> details = (Map<String, String>) dsInfos.get("details");
 
         // Validate and set capacity
         if (capacityBytes == null || capacityBytes <= 0) {

ui/src/views/infra/AddPrimaryStorage.vue

@@ -931,6 +931,12 @@ export default {
           params['details[0].api_username'] = values.flashArrayUsername
           params['details[0].api_password'] = values.flashArrayPassword
           url = values.flashArrayURL
+        } else if (values.provider === 'ONTAP') {
+          params['details[0].username'] = values.ontapUsername
+          params['details[0].password'] = values.ontapPassword
+          params['details[0].svm_name'] = values.ontapSvmName
+          values.managed = true
+          url = this.ontapURL(values.ontapIP)
         }
 
         if (values.provider === 'Linstor' || values.protocol === 'Linstor') {
@@ -941,12 +947,7 @@ export default {
             params.capacityIops = values.capacityIops.split(',').join('')
           }
         }
-        console.log('URL value is ::: ', url)
-        if (values.provider === 'ONTAP') {
-          console.log('URL value is getting set ::: ', url)
-          url = this.ontapURL(values.ontapIP)
-        }
-        console.log('URL value is now ::: ', url)
+
         params.url = url
         if (values.provider !== 'DefaultPrimary' && values.provider !== 'PowerFlex') {
           if (values.managed) {
@@ -970,10 +971,6 @@ export default {
             values.powerflexGatewayPassword, values.powerflexStoragePool)
         }
 
-        if (values.provider === 'ONTAP') {
-          params.url = this.ontapURL(values.ontapIP)
-        }
-
         if (this.selectedTags.length > 0) {
           params.tags = this.selectedTags.join()
         }