vm instance creation test11

Author: Srivastava, Piyush

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

@@ -137,25 +137,27 @@ private String createCloudStackVolumeForTypeVolume(DataStore dataStore, DataObje
             // CRITICAL: Set poolType at VOLUME level
             // This tells KVM to use LibvirtStorageAdaptor (NFS) instead of IscsiAdmStorageAdaptor
             volume.setPoolType(Storage.StoragePoolType.NetworkFilesystem);
-            // Use UUID only (KVM's default behavior for managed NFS)
-            // KVM's getPhysicalDisk() is hardcoded to use UUID, not the path we set
+            // Create unique volume name: name_uuid format for readability
+            // Example: ROOT-22_e4cf5d79-3a5f-4618-b3c5-f727ff3d1603
+            String volumeName = ((VolumeInfo) dataObject).getName();
             String volumeUuid = ((VolumeInfo) dataObject).getUuid();
-            // Set path to UUID (KVM will use this)
-            volume.setPath(volumeUuid);
+            String uniqueVolumeName = volumeName + "_" + volumeUuid;
+            // Set path to the unique name so KVM can find it later
+            // This is critical - the path is what KVM uses to locate the file
+            volume.setPath(uniqueVolumeName);
             volume.setFolder("");  // No folder for flat structure
             volume.setPoolId(dataStore.getId());
             volumeDao.update(volume.getId(), volume);
-            s_logger.info("=== ONTAP Managed NFS Volume (Flat Structure - UUID only) ===");
-            s_logger.info("Volume Name: {}", volume.getName());
+            s_logger.info("=== ONTAP Managed NFS Volume (Flat Structure) ===");
+            s_logger.info("Volume Name: {}", volumeName);
             s_logger.info("Volume UUID: {}", volumeUuid);
-            s_logger.info("Volume Path: {} (UUID-based, KVM default)", volumeUuid);
+            s_logger.info("Volume Path: {} (name_uuid format)", uniqueVolumeName);
             s_logger.info("Volume PoolType: {}", volume.getPoolType());
-            s_logger.info("Returning null - hypervisor will create: /mnt/<pool>/{}.qcow2", volumeUuid);
-            s_logger.info("Note: KVM's LibvirtStoragePool.getPhysicalDisk() is hardcoded to use UUID");
+            s_logger.info("Hypervisor will create: /mnt/<pool>/{}.qcow2", uniqueVolumeName);
             s_logger.info("================================================");
-            // CRITICAL: Return null so CloudStack triggers KVM to create the volume!
-            // KVM will create file as: /mnt/pool/<uuid>.qcow2
-            return null;
+            // CRITICAL: Return the unique name (not null!)
+            // This tells CloudStack what path was created, so KVM can find it later
+            return uniqueVolumeName;
         }
         // For iSCSI and other protocols, use ONTAP REST API
         StorageStrategy storageStrategy = getStrategyByStoragePoolDetails(details);