Commit 16

Author: Srivastava, Piyush

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/OntapNfsStorageAdaptor.java

@@ -171,12 +171,9 @@ public boolean connectPhysicalDisk(String volumeUuid, KVMStoragePool pool, Map<S
             }
 
             logger.info("Successfully mounted ONTAP NFS volume: " + nfsServer + ":" + junctionPath + " at " + mountPoint);
-            
             // Store the mapping so other methods can find the correct mount point
             volumeToMountPointMap.put(volumeUuid, mountPoint);
-            
             return true;
-
         } catch (Exception e) {
             logger.error("Exception mounting ONTAP NFS volume: " + volumeUuid, e);
             return false;
@@ -215,27 +212,21 @@ public boolean disconnectPhysicalDisk(String volumeUuid, KVMStoragePool pool) {
             logger.info("Volume not mounted, nothing to disconnect: " + mountPoint);
             return true;
         }
-
         try {
             // Unmount: umount <mountPoint>
             String umountCmd = "umount " + mountPoint;
             Script script = new Script("/bin/bash", 60000, logger);
             script.add("-c");
             script.add(umountCmd);
             String result = script.execute();
-
             if (result != null) {
                 logger.warn("Failed to unmount ONTAP NFS volume: " + mountPoint + ", error: " + result);
                 return false;
             }
-
             logger.info("Successfully unmounted ONTAP NFS volume: " + mountPoint);
-            
             // Remove from mapping
             volumeToMountPointMap.remove(volumeUuid);
-            
             return true;
-
         } catch (Exception e) {
             logger.error("Exception unmounting ONTAP NFS volume: " + volumeUuid, e);
             return false;