Rename ambiguous global setting: enable.kvm.host.auto.enable.disable

Author: Pearl1594

engine/components-api/src/main/java/com/cloud/agent/AgentManager.java

@@ -39,8 +39,8 @@
 public interface AgentManager {
     static final ConfigKey<Integer> Wait = new ConfigKey<Integer>("Advanced", Integer.class, "wait", "1800", "Time in seconds to wait for control commands to return",
             true);
-    ConfigKey<Boolean> EnableKVMAutoEnableDisable = new ConfigKey<>(Boolean.class,
-                    "enable.kvm.host.auto.enable.disable",
+    ConfigKey<Boolean> KVMAutoEnableDisable = new ConfigKey<>(Boolean.class,
+                    "enable.kvm.host.auto.enable",
                     "Advanced",
                     "false",
                     "(KVM only) Enable Auto Disable/Enable KVM hosts in the cluster " +

engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java

@@ -1268,9 +1268,9 @@ private void processHostHealthCheckResult(Boolean hostHealthCheckResult, long ho
                 logger.error("Unable to find host with ID: {}", hostId);
                 return;
             }
-            if (!BooleanUtils.toBoolean(EnableKVMAutoEnableDisable.valueIn(host.getClusterId()))) {
-                logger.debug("{} is disabled for the cluster {}, cannot process the health check result " +
-                        "received for the host {}", EnableKVMAutoEnableDisable.key(), host.getClusterId(), host.getName());
+            if (!BooleanUtils.toBoolean(KVMAutoEnableDisable.valueIn(host.getClusterId()))) {
+                logger.debug(String.format("%s is disabled for the cluster %s, cannot process the health check result " +
+                        "received for the host %s", KVMAutoEnableDisable.key(), host.getClusterId(), host.getName()));
                 return;
             }
 
@@ -1802,7 +1802,7 @@ public String getConfigComponentName() {
     @Override
     public ConfigKey<?>[] getConfigKeys() {
         return new ConfigKey<?>[] { CheckTxnBeforeSending, Workers, Port, Wait, AlertWait, DirectAgentLoadSize,
-                DirectAgentPoolSize, DirectAgentThreadCap, EnableKVMAutoEnableDisable, ReadyCommandWait };
+                DirectAgentPoolSize, DirectAgentThreadCap, KVMAutoEnableDisable, ReadyCommandWait };
     }
 
     protected class SetHostParamsListener implements Listener {

server/src/main/java/com/cloud/resource/ResourceManagerImpl.java

@@ -1841,7 +1841,7 @@ private void handleAutoEnableDisableKVMHost(boolean autoEnableDisableKVMSetting,
                     Boolean.parseBoolean(hostDetail.getValue()) && resourceEvent == ResourceState.Event.Disable) {
                 logger.info(String.format("The setting %s is enabled but the host %s is manually set into %s state," +
                                 "ignoring future auto enabling of the host based on health check results",
-                        AgentManager.EnableKVMAutoEnableDisable.key(), host.getName(), resourceEvent));
+                        AgentManager.KVMAutoEnableDisable.key(), host.getName(), resourceEvent));
                 hostDetail.setValue(Boolean.FALSE.toString());
                 _hostDetailsDao.update(hostDetail.getId(), hostDetail);
             } else if (hostDetail == null) {
@@ -1853,7 +1853,7 @@ private void handleAutoEnableDisableKVMHost(boolean autoEnableDisableKVMSetting,
     }
     private boolean updateHostAllocationState(HostVO host, String allocationState,
                                            boolean isUpdateFromHostHealthCheck) throws NoTransitionException {
-        boolean autoEnableDisableKVMSetting = AgentManager.EnableKVMAutoEnableDisable.valueIn(host.getClusterId()) &&
+        boolean autoEnableDisableKVMSetting = AgentManager.KVMAutoEnableDisable.valueIn(host.getClusterId()) &&
                 host.getHypervisorType() == HypervisorType.KVM;
         ResourceState.Event resourceEvent = getResourceEventFromAllocationStateString(allocationState);
         DetailVO hostDetail = _hostDetailsDao.findDetail(host.getId(), ApiConstants.AUTO_ENABLE_KVM_HOST);
@@ -1867,7 +1867,7 @@ private boolean updateHostAllocationState(HostVO host, String allocationState,
         if (isAutoEnableAttemptForADisabledHost(autoEnableDisableKVMSetting, isUpdateFromHostHealthCheck, hostDetail, resourceEvent)) {
             logger.debug(String.format("The setting '%s' is enabled and the health check succeeds on the host, " +
                             "but the host has been manually disabled previously, ignoring auto enabling",
-                    AgentManager.EnableKVMAutoEnableDisable.key()));
+                    AgentManager.KVMAutoEnableDisable.key()));
             return false;
         }
 
@@ -1981,7 +1981,7 @@ private void sendAlertAndAnnotationForAutoEnableDisableKVMHostFeature(HostVO hos
                                                                           boolean isUpdateFromHostHealthCheck,
                                                                           boolean isUpdateHostAllocation, String annotation) {
         boolean isAutoEnableDisableKVMSettingEnabled = host.getHypervisorType() == HypervisorType.KVM &&
-                AgentManager.EnableKVMAutoEnableDisable.valueIn(host.getClusterId());
+                AgentManager.KVMAutoEnableDisable.valueIn(host.getClusterId());
         if (!isAutoEnableDisableKVMSettingEnabled) {
             if (StringUtils.isNotBlank(annotation)) {
                 annotationService.addAnnotation(annotation, AnnotationService.EntityType.HOST, host.getUuid(), true);
@@ -2005,7 +2005,7 @@ private void sendAlertAndAnnotationForAutoEnableDisableKVMHostFeature(HostVO hos
                     host.getPodId(), msg, msg);
         } else {
             msg += String.format("is %s despite the setting '%s' is enabled for the cluster %s",
-                    isEventEnable ? "enabled" : "disabled", AgentManager.EnableKVMAutoEnableDisable.key(),
+                    isEventEnable ? "enabled" : "disabled", AgentManager.KVMAutoEnableDisable.key(),
                     host.getClusterId());
             if (StringUtils.isNotBlank(annotation)) {
                 msg += String.format(", reason: %s", annotation);