vm instance creation test7

Author: Srivastava, Piyush

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

@@ -293,14 +293,13 @@ private String createManagedNfsVolume(DataStore dataStore, DataObject dataObject
         String junctionPath = storagePool.getPath();
 
         // Update volume metadata in CloudStack database
-        volume.setPoolType(Storage.StoragePoolType.ManagedNFS);
+        // Use NetworkFilesystem (not ManagedNFS) - matches pool type set during pool creation
+        volume.setPoolType(Storage.StoragePoolType.NetworkFilesystem);
         volume.setPoolId(dataStore.getId());
         volume.setPath(volumeUuid);  // Filename for qcow2 file
 
-        // CRITICAL: For ManagedNFS, _iScsiName must be the volume UUID, NOT the junction path
-        // CloudStack uses _iScsiName as MANAGED_STORE_TARGET -> volPath -> libvirt pool name
-        // Libvirt pool names cannot contain '/', so we use the UUID
-        // The junction path comes from storage_pool_details.mountpoint (set in getStoreTO)
+        // For NetworkFilesystem with managed=true, _iScsiName should be set to volume UUID
+        // This maintains compatibility with managed storage behavior
         volume.set_iScsiName(volumeUuid);
 
         volumeDao.update(volume.getId(), volume);

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

@@ -192,7 +192,10 @@ public DataStore initialize(Map<String, Object> dsInfos) {
         ProtocolType protocol = ProtocolType.valueOf(details.get(Constants.PROTOCOL));
         switch (protocol) {
             case NFS:
-                parameters.setType(Storage.StoragePoolType.ManagedNFS);
+                // Use NetworkFilesystem (not ManagedNFS) with managed=true
+                // This routes to LibvirtStorageAdaptor which has full qemu-img support
+                // Same pattern as CloudByte/Elastistor plugin
+                parameters.setType(Storage.StoragePoolType.NetworkFilesystem);
                 // Path should be just the NFS export path (junction path), NOT host:path
                 // CloudStack will construct the full mount path as: hostAddress + ":" + path
                 path = "/" + storagePoolName;