You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by da...@apache.org on 2025/08/29 13:13:06 UTC
(cloudstack) branch 4.20 updated: kvm: fix vm deployment with direct-download iso (#11532)
This is an automated email from the ASF dual-hosted git repository.
dahn pushed a commit to branch 4.20
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/4.20 by this push:
new 762f75c041d kvm: fix vm deployment with direct-download iso (#11532)
762f75c041d is described below
commit 762f75c041d7c14c9d4846bf095381778735a171
Author: Abhishek Kumar <ab...@gmail.com>
AuthorDate: Fri Aug 29 18:42:59 2025 +0530
kvm: fix vm deployment with direct-download iso (#11532)
---
.../java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java b/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java
index cdb523a50fd..4ea8d1e80fb 100644
--- a/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java
+++ b/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java
@@ -84,9 +84,8 @@ import org.apache.commons.lang3.BooleanUtils;
import org.apache.commons.lang3.StringUtils;
import org.apache.commons.lang3.builder.ToStringBuilder;
import org.apache.commons.lang3.builder.ToStringStyle;
-import org.apache.logging.log4j.Logger;
import org.apache.logging.log4j.LogManager;
-
+import org.apache.logging.log4j.Logger;
import org.libvirt.Connect;
import org.libvirt.Domain;
import org.libvirt.DomainInfo;
@@ -2476,7 +2475,9 @@ public class KVMStorageProcessor implements StorageProcessor {
if (template != null) {
templatePath = template.getPath();
}
- if (StringUtils.isEmpty(templatePath)) {
+ if (ImageFormat.ISO.equals(cmd.getFormat())) {
+ logger.debug("Skipping template validations as image format is {}", cmd.getFormat());
+ } else if (StringUtils.isEmpty(templatePath)) {
logger.warn("Skipped validation whether downloaded file is QCOW2 for template {}, due to downloaded template path is empty", template.getName());
} else if (!new File(templatePath).exists()) {
logger.warn("Skipped validation whether downloaded file is QCOW2 for template {}, due to downloaded template path is not valid: {}", template.getName(), templatePath);