From 12f2d804ebcaa62c790b1d9cd699b080320902d6 Mon Sep 17 00:00:00 2001
From: Abhisar Sinha <63767682+abh1sar@users.noreply.github.com>
Date: Wed, 16 Jul 2025 14:03:18 +0530
Subject: [PATCH 1/5] Guard OS type update for iso/template with existing vms

---
 .../apache/cloudstack/api/ApiConstants.java   |  1 +
 .../api/BaseUpdateTemplateOrIsoCmd.java       |  8 ++++++++
 .../cloud/template/TemplateManagerImpl.java   |  6 ++++++
 ui/public/locales/en.json                     |  1 +
 ui/src/views/image/UpdateISO.vue              | 19 +++++++++++++++++--
 ui/src/views/image/UpdateTemplate.vue         | 18 +++++++++++++++++-
 6 files changed, 50 insertions(+), 3 deletions(-)

diff --git a/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java b/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
index 22e7e8075020..9bbfe25df2af 100644
--- a/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
+++ b/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
@@ -220,6 +220,7 @@ public class ApiConstants {
     public static final String FORCED_DESTROY_LOCAL_STORAGE = "forcedestroylocalstorage";
     public static final String FORCE_DELETE_HOST = "forcedeletehost";
     public static final String FORCE_MS_TO_IMPORT_VM_FILES = "forcemstoimportvmfiles";
+    public static final String FORCE_UPDATE_OS_TYPE = "forceupdateostype";
     public static final String FORMAT = "format";
     public static final String FOR_VIRTUAL_NETWORK = "forvirtualnetwork";
     public static final String FOR_SYSTEM_VMS = "forsystemvms";
diff --git a/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java b/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
index 9a8282df1121..5be61d83eed4 100644
--- a/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
+++ b/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
@@ -51,6 +51,10 @@ public abstract class BaseUpdateTemplateOrIsoCmd extends BaseCmd {
                description = "the ID of the OS type that best represents the OS of this image.")
     private Long osTypeId;
 
+    @Parameter(name = ApiConstants.FORCE_UPDATE_OS_TYPE, type = CommandType.BOOLEAN, description = "Force OS type update. Warning: Updating OS type will update the guest OS configuration" +
+            "for all the existing VMs deployed with this template/iso, which may affect their behavior.")
+    private Boolean forceUpdateOsType;
+
     @Parameter(name = ApiConstants.FORMAT, type = CommandType.STRING, description = "the format for the image")
     private String format;
 
@@ -112,6 +116,10 @@ public Long getOsTypeId() {
         return osTypeId;
     }
 
+    public Boolean getForceUpdateOsType() {
+        return forceUpdateOsType;
+    }
+
     public Boolean getPasswordEnabled() {
         return passwordEnabled;
     }
diff --git a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
index 6759144837d5..672b3f009482 100755
--- a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
@@ -2238,6 +2238,12 @@ private VMTemplateVO updateTemplateOrIso(BaseUpdateTemplateOrIsoCmd cmd) {
                 sc.addAnd("state", SearchCriteria.Op.NEQ, State.Expunging);
                 List<VMInstanceVO> vms = _vmInstanceDao.search(sc, null);
                 if (vms != null && !vms.isEmpty()) {
+                    if (!Boolean.TRUE.equals(cmd.getForceUpdateOsType())) {
+                        String message = String.format("Updating OS type will update the guest OS configuration " +
+                        "for all of the %d VM(s) deployed with this Template/ISO, which may affect their behavior. " +
+                        "To proceed, please set the 'forceupdateostype' parameter to true.", vms.size());
+                        throw new InvalidParameterValueException(message);
+                    }
                     for (VMInstanceVO vm: vms) {
                         vm.setGuestOSId(guestOSId);
                         _vmInstanceDao.update(vm.getId(), vm);
diff --git a/ui/public/locales/en.json b/ui/public/locales/en.json
index b649f6914bb9..028d32eccc67 100644
--- a/ui/public/locales/en.json
+++ b/ui/public/locales/en.json
@@ -1038,6 +1038,7 @@
 "label.forbidden": "Forbidden",
 "label.forced": "Force",
 "label.force.ms.to.import.vm.files": "Enable to force OVF Download via Management Server. Disable to use KVM Host ovftool (if installed)",
+"label.force.update.os.type": "Force update OS type",
 "label.force.stop": "Force stop",
 "label.force.reboot": "Force reboot",
 "label.forceencap": "Force UDP encapsulation of ESP packets",
diff --git a/ui/src/views/image/UpdateISO.vue b/ui/src/views/image/UpdateISO.vue
index 856038f2d5f9..15d57458ff6a 100644
--- a/ui/src/views/image/UpdateISO.vue
+++ b/ui/src/views/image/UpdateISO.vue
@@ -67,6 +67,13 @@
             </a-select-option>
           </a-select>
         </a-form-item>
+        <a-form-item name="forceupdateostype" ref="forceupdateostype" v-if="hasOstypeidChanged()">
+          <template #label>
+            <tooltip-label :title="$t('label.force.update.os.type')" :tooltip="apiParams.forceupdateostype.description"/>
+          </template>
+          <a-switch v-model:checked="form.forceupdateostype" />
+        </a-form-item>
+
         <a-form-item name="isdynamicallyscalable" ref="isdynamicallyscalable">
           <template #label>
             <tooltip-label :title="$t('label.isdynamicallyscalable')" :tooltip="apiParams.isdynamicallyscalable.description"/>
@@ -172,7 +179,8 @@ export default {
       userdataid: null,
       userdatapolicy: null,
       userdatapolicylist: {},
-      architectureTypes: {}
+      architectureTypes: {},
+      originalOstypeid: null
     }
   },
   beforeCreate () {
@@ -195,7 +203,7 @@ export default {
         displaytext: [{ required: true, message: this.$t('message.error.required.input') }],
         ostypeid: [{ required: true, message: this.$t('message.error.select') }]
       })
-      const resourceFields = ['name', 'displaytext', 'passwordenabled', 'isdynamicallyscalable', 'ostypeid', 'userdataid', 'userdatapolicy']
+      const resourceFields = ['name', 'displaytext', 'passwordenabled', 'isdynamicallyscalable', 'ostypeid', 'forceupdateostype', 'userdataid', 'userdatapolicy']
 
       for (var field of resourceFields) {
         var fieldValue = this.resource[field]
@@ -207,6 +215,10 @@ export default {
             case 'userdatapolicy':
               this.userdatapolicy = fieldValue
               break
+            case 'ostypeid':
+              this.form[field] = fieldValue
+              this.originalOstypeid = fieldValue
+              break
             default:
               this.form[field] = fieldValue
               break
@@ -214,6 +226,9 @@ export default {
         }
       }
     },
+    hasOstypeidChanged () {
+      return this.form.ostypeid !== this.originalOstypeid
+    },
     fetchData () {
       this.fetchOsTypes()
       this.architectureTypes.opts = this.$fetchCpuArchitectureTypes()
diff --git a/ui/src/views/image/UpdateTemplate.vue b/ui/src/views/image/UpdateTemplate.vue
index 75a41accecba..2e6563bc3c10 100644
--- a/ui/src/views/image/UpdateTemplate.vue
+++ b/ui/src/views/image/UpdateTemplate.vue
@@ -103,6 +103,14 @@
               </a-select>
             </a-form-item>
           </a-col>
+          <a-col :md="24" :lg="24" v-if="hasOstypeidChanged()">
+            <a-form-item name="forceupdateostype" ref="forceupdateostype">
+              <template #label>
+                <tooltip-label :title="$t('label.force.update.os.type')" :tooltip="apiParams.forceupdateostype.description"/>
+              </template>
+              <a-switch v-model:checked="form.forceupdateostype" />
+            </a-form-item>
+          </a-col>
         </a-row>
         <a-row :gutter="12">
           <a-col :md="24" :lg="12">
@@ -251,7 +259,8 @@ export default {
       userdataid: null,
       userdatapolicy: null,
       userdatapolicylist: {},
-      architectureTypes: {}
+      architectureTypes: {},
+      originalOstypeid: null
     }
   },
   beforeCreate () {
@@ -295,6 +304,10 @@ export default {
             case 'userdatapolicy':
               this.userdatapolicy = fieldValue
               break
+            case 'ostypeid':
+              this.form[field] = fieldValue
+              this.originalOstypeid = fieldValue
+              break
             default:
               this.form[field] = fieldValue
               break
@@ -546,6 +559,9 @@ export default {
       }).finally(() => {
         this.loading = false
       })
+    },
+    hasOstypeidChanged () {
+      return this.form.ostypeid !== this.originalOstypeid
     }
   }
 }

From b317abfb3f2bf077313d7f078bbdf197c19672ff Mon Sep 17 00:00:00 2001
From: Abhisar Sinha <63767682+abh1sar@users.noreply.github.com>
Date: Wed, 16 Jul 2025 14:20:03 +0530
Subject: [PATCH 2/5] fix identation

---
 .../org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java | 4 ++--
 .../src/main/java/com/cloud/template/TemplateManagerImpl.java | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java b/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
index 5be61d83eed4..9cb76d6f88f4 100644
--- a/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
+++ b/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
@@ -51,8 +51,8 @@ public abstract class BaseUpdateTemplateOrIsoCmd extends BaseCmd {
                description = "the ID of the OS type that best represents the OS of this image.")
     private Long osTypeId;
 
-    @Parameter(name = ApiConstants.FORCE_UPDATE_OS_TYPE, type = CommandType.BOOLEAN, description = "Force OS type update. Warning: Updating OS type will update the guest OS configuration" +
-            "for all the existing VMs deployed with this template/iso, which may affect their behavior.")
+    @Parameter(name = ApiConstants.FORCE_UPDATE_OS_TYPE, type = CommandType.BOOLEAN, since = "4.21", description = "Force OS type update. Warning: Updating OS type will " +
+            "update the guest OS configuration for all the existing VMs deployed with this template/iso, which may affect their behavior.")
     private Boolean forceUpdateOsType;
 
     @Parameter(name = ApiConstants.FORMAT, type = CommandType.STRING, description = "the format for the image")
diff --git a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
index 672b3f009482..e47374b56325 100755
--- a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
@@ -2240,8 +2240,8 @@ private VMTemplateVO updateTemplateOrIso(BaseUpdateTemplateOrIsoCmd cmd) {
                 if (vms != null && !vms.isEmpty()) {
                     if (!Boolean.TRUE.equals(cmd.getForceUpdateOsType())) {
                         String message = String.format("Updating OS type will update the guest OS configuration " +
-                        "for all of the %d VM(s) deployed with this Template/ISO, which may affect their behavior. " +
-                        "To proceed, please set the 'forceupdateostype' parameter to true.", vms.size());
+                            "for all of the %d VM(s) deployed with this Template/ISO, which may affect their behavior. " +
+                            "To proceed, please set the 'forceupdateostype' parameter to true.", vms.size());
                         throw new InvalidParameterValueException(message);
                     }
                     for (VMInstanceVO vm: vms) {

From fb3fc51b726c26356663844aaaa491b9ad9ed4e0 Mon Sep 17 00:00:00 2001
From: Abhisar Sinha <63767682+abh1sar@users.noreply.github.com>
Date: Wed, 16 Jul 2025 14:21:16 +0530
Subject: [PATCH 3/5] rename vm -> instance

---
 .../org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java   | 2 +-
 .../src/main/java/com/cloud/template/TemplateManagerImpl.java   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java b/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
index 9cb76d6f88f4..8489bf05ec78 100644
--- a/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
+++ b/api/src/main/java/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java
@@ -52,7 +52,7 @@ public abstract class BaseUpdateTemplateOrIsoCmd extends BaseCmd {
     private Long osTypeId;
 
     @Parameter(name = ApiConstants.FORCE_UPDATE_OS_TYPE, type = CommandType.BOOLEAN, since = "4.21", description = "Force OS type update. Warning: Updating OS type will " +
-            "update the guest OS configuration for all the existing VMs deployed with this template/iso, which may affect their behavior.")
+            "update the guest OS configuration for all the existing Instances deployed with this template/iso, which may affect their behavior.")
     private Boolean forceUpdateOsType;
 
     @Parameter(name = ApiConstants.FORMAT, type = CommandType.STRING, description = "the format for the image")
diff --git a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
index e47374b56325..8d4c64b4ceee 100755
--- a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
@@ -2240,7 +2240,7 @@ private VMTemplateVO updateTemplateOrIso(BaseUpdateTemplateOrIsoCmd cmd) {
                 if (vms != null && !vms.isEmpty()) {
                     if (!Boolean.TRUE.equals(cmd.getForceUpdateOsType())) {
                         String message = String.format("Updating OS type will update the guest OS configuration " +
-                            "for all of the %d VM(s) deployed with this Template/ISO, which may affect their behavior. " +
+                            "for all of the %d Instance(s) deployed with this Template/ISO, which may affect their behavior. " +
                             "To proceed, please set the 'forceupdateostype' parameter to true.", vms.size());
                         throw new InvalidParameterValueException(message);
                     }

From 1bc93d3bef9f466025ba183333573e349a27a927 Mon Sep 17 00:00:00 2001
From: Abhisar Sinha <63767682+abh1sar@users.noreply.github.com>
Date: Mon, 21 Jul 2025 16:52:12 +0530
Subject: [PATCH 4/5] force update iso/template as true by default via api

---
 .../src/main/java/com/cloud/template/TemplateManagerImpl.java   | 2 +-
 ui/src/views/image/UpdateISO.vue                                | 1 +
 ui/src/views/image/UpdateTemplate.vue                           | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
index 8d4c64b4ceee..d8e90cd3d05b 100755
--- a/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/main/java/com/cloud/template/TemplateManagerImpl.java
@@ -2238,7 +2238,7 @@ private VMTemplateVO updateTemplateOrIso(BaseUpdateTemplateOrIsoCmd cmd) {
                 sc.addAnd("state", SearchCriteria.Op.NEQ, State.Expunging);
                 List<VMInstanceVO> vms = _vmInstanceDao.search(sc, null);
                 if (vms != null && !vms.isEmpty()) {
-                    if (!Boolean.TRUE.equals(cmd.getForceUpdateOsType())) {
+                    if (Boolean.FALSE.equals(cmd.getForceUpdateOsType())) {
                         String message = String.format("Updating OS type will update the guest OS configuration " +
                             "for all of the %d Instance(s) deployed with this Template/ISO, which may affect their behavior. " +
                             "To proceed, please set the 'forceupdateostype' parameter to true.", vms.size());
diff --git a/ui/src/views/image/UpdateISO.vue b/ui/src/views/image/UpdateISO.vue
index 15d57458ff6a..f4b46665352a 100644
--- a/ui/src/views/image/UpdateISO.vue
+++ b/ui/src/views/image/UpdateISO.vue
@@ -310,6 +310,7 @@ export default {
           if (!this.isValidValueForKey(values, key)) continue
           params[key] = values[key]
         }
+        params.forceupdateostype = this.form.forceupdateostype || false
         postAPI('updateIso', params).then(json => {
           if (this.userdataid !== null) {
             this.linkUserdataToTemplate(this.userdataid, json.updateisoresponse.iso.id, this.userdatapolicy)
diff --git a/ui/src/views/image/UpdateTemplate.vue b/ui/src/views/image/UpdateTemplate.vue
index 2e6563bc3c10..117b53bbc282 100644
--- a/ui/src/views/image/UpdateTemplate.vue
+++ b/ui/src/views/image/UpdateTemplate.vue
@@ -523,6 +523,7 @@ export default {
           }
           params[key] = values[key]
         }
+        params.forceupdateostype = this.form.forceupdateostype || false
         postAPI('updateTemplate', params).then(json => {
           if (this.userdataid !== null) {
             this.linkUserdataToTemplate(this.userdataid, json.updatetemplateresponse.template.id, this.userdatapolicy)

From 356c03c68a1f2c29a14da0e4c823b1e1cc99f41e Mon Sep 17 00:00:00 2001
From: Abhisar Sinha <63767682+abh1sar@users.noreply.github.com>
Date: Mon, 21 Jul 2025 18:59:09 +0530
Subject: [PATCH 5/5] add missing message.success.update.iso label

---
 ui/public/locales/en.json | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ui/public/locales/en.json b/ui/public/locales/en.json
index 028d32eccc67..d065023cc974 100644
--- a/ui/public/locales/en.json
+++ b/ui/public/locales/en.json
@@ -3692,6 +3692,7 @@
 "message.success.update.ipaddress": "Successfully updated IP address",
 "message.success.update.iprange": "Successfully updated IP range",
 "message.success.update.ipv4.subnet": "Successfully updated IPv4 subnet",
+"message.success.update.iso": "Successfully updated ISO",
 "message.success.update.kubeversion": "Successfully updated Kubernetes supported version",
 "message.success.update.network": "Successfully updated Network",
 "message.success.update.template": "Successfully updated Template",