kvm: consider provisioning type for local data volumes (#9141)

shwstppr · web-flow · commit 10f4de031813 · 2024-06-10T11:38:31.000+03:00
Fixes #8644 Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
diff --git a/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java b/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
@@ -842,7 +842,7 @@ private KVMPhysicalDisk createPhysicalDiskByQemuImg(String name, KVMStoragePool
         destFile.setFormat(format);
         destFile.setSize(size);
         Map<String, String> options = new HashMap<String, String>();
-        if (pool.getType() == StoragePoolType.NetworkFilesystem){
+        if (List.of(StoragePoolType.NetworkFilesystem, StoragePoolType.Filesystem).contains(pool.getType())) {
             options.put("preallocation", QemuImg.PreallocationType.getPreallocationType(provisioningType).toString());
         }
 

Original file line number	Diff line number	Diff line change
`@@ -842,7 +842,7 @@ private KVMPhysicalDisk createPhysicalDiskByQemuImg(String name, KVMStoragePool`
`842`	`842`	`destFile.setFormat(format);`
`843`	`843`	`destFile.setSize(size);`
`844`	`844`	`Map<String, String> options = new HashMap<String, String>();`
`845`		`- if (pool.getType() == StoragePoolType.NetworkFilesystem){`
	`845`	`+ if (List.of(StoragePoolType.NetworkFilesystem, StoragePoolType.Filesystem).contains(pool.getType())) {`
`846`	`846`	`options.put("preallocation", QemuImg.PreallocationType.getPreallocationType(provisioningType).toString());`
`847`	`847`	`}`
`848`	`848`