CSTACKEX-46 Change log level

Author: Gupta, Surya

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/UnifiedSANStrategy.java

@@ -107,7 +107,8 @@ void deleteCloudStackVolume(CloudStackVolume cloudstackVolume) {
 
     @Override
     public CloudStackVolume getCloudStackVolume(Map<String, String> values) {
-        s_logger.info("getCloudStackVolume : fetching Lun with params {} ", values);
+        s_logger.info("getCloudStackVolume : fetching Lun");
+        s_logger.debug("getCloudStackVolume : fetching Lun with params {} ", values);
         if (values == null || values.isEmpty()) {
             s_logger.error("getCloudStackVolume: get Lun failed. Invalid request: {}", values);
             throw new CloudRuntimeException("getCloudStackVolume : get Lun Failed, invalid request");
@@ -144,9 +145,10 @@ public CloudStackVolume getCloudStackVolume(Map<String, String> values) {
     @Override
     public AccessGroup createAccessGroup(AccessGroup accessGroup) {
         s_logger.info("createAccessGroup : Create Igroup");
-        String igroupName = "unknown";
-        if (accessGroup == null) {
-            throw new CloudRuntimeException("createAccessGroup : Failed to create Igroup, invalid accessGroup object passed");
+        s_logger.debug("createAccessGroup : Creating Igroup with access group request {} ", accessGroup);
+        if (accessGroup == null || accessGroup.getIgroup() == null) {
+            s_logger.error("createAccessGroup: Igroup creation failed. Invalid request: {}", accessGroup);
+            throw new CloudRuntimeException("createAccessGroup : Failed to create Igroup, invalid request");
         }
         try {
             // Get StoragePool details
@@ -340,7 +342,8 @@ public AccessGroup updateAccessGroup(AccessGroup accessGroup) {
     }
 
     public AccessGroup getAccessGroup(Map<String, String> values) {
-        s_logger.info("getAccessGroup : fetching Igroup with params {} ", values);
+        s_logger.info("getAccessGroup : fetch Igroup");
+        s_logger.debug("getAccessGroup : fetching Igroup with params {} ", values);
         if (values == null || values.isEmpty()) {
             s_logger.error("getAccessGroup: get Igroup failed. Invalid request: {}", values);
             throw new CloudRuntimeException("getAccessGroup : get Igroup Failed, invalid request");
@@ -372,7 +375,8 @@ public AccessGroup getAccessGroup(Map<String, String> values) {
     }
 
     public void enableLogicalAccess(Map<String, String> values) {
-        s_logger.info("enableLogicalAccess : Creating LunMap with values {} ", values);
+        s_logger.info("enableLogicalAccess : Create LunMap");
+        s_logger.debug("enableLogicalAccess : Creating LunMap with values {} ", values);
         LunMap lunMapRequest = new LunMap();
         String svmName = values.get(Constants.SVM_DOT_NAME);
         String lunName = values.get(Constants.LUN_DOT_NAME);
@@ -400,7 +404,8 @@ public void enableLogicalAccess(Map<String, String> values) {
     }
 
     public void disableLogicalAccess(Map<String, String> values) {
-        s_logger.info("disableLogicalAccess : Deleting LunMap with values {} ", values);
+        s_logger.info("disableLogicalAccess : Delete LunMap");
+        s_logger.debug("disableLogicalAccess : Deleting LunMap with values {} ", values);
         String lunUUID = values.get(Constants.LUN_DOT_UUID);
         String igroupUUID = values.get(Constants.IGROUP_DOT_UUID);
         if(lunUUID == null || igroupUUID == null || lunUUID.isEmpty() || igroupUUID.isEmpty()) {