vm instance creation test10

Author: Srivastava, Piyush

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

@@ -137,27 +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 unique volume name combining CloudStack volume name and UUID
-            // This creates a unique filename directly in NFS root (flat structure)
+            // 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();
             String uniqueVolumeName = volumeName + "_" + volumeUuid;
-            // For managed NFS, set the volume name to the unique name
-            // KVM will create qcow2 file directly in NFS root: /mnt/pool/ROOT-21_uuid.qcow2
-            volume.setName(uniqueVolumeName);
+            // 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(volumeUuid);  // Store UUID for reference
+            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) ===");
-            s_logger.info("Volume Name: {} (used by KVM for file creation)", uniqueVolumeName);
-            s_logger.info("Volume Path: {}", volume.getPath());
+            s_logger.info("Volume Name: {}", volumeName);
             s_logger.info("Volume UUID: {}", volumeUuid);
+            s_logger.info("Volume Path: {} (name_uuid format)", uniqueVolumeName);
             s_logger.info("Volume PoolType: {}", volume.getPoolType());
             s_logger.info("Hypervisor will create: /mnt/<pool>/{}.qcow2", uniqueVolumeName);
             s_logger.info("================================================");
-            // Return null - let hypervisor create the qcow2 file directly in NFS root
-            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);