Merge pull request #785 from sjyu1/ablestack-europa

20260305일자 europa 병합 오류 수정

Author: jschoiRR

api/src/main/java/com/cloud/vm/VmDetailConstants.java

@@ -134,4 +134,5 @@ public interface VmDetailConstants {
     String EXTERNAL_DETAIL_PREFIX = "External:";
     String CLOUDSTACK_VM_DETAILS = "cloudstack.vm.details";
     String CLOUDSTACK_VLAN = "cloudstack.vlan";
+    String KVM_GUEST_OS_MACHINE_TYPE = "kvm.guest.os.machine.type";
 }

api/src/main/java/org/apache/cloudstack/api/command/admin/ha/DisableHAForClusterCmd.java

@@ -95,7 +95,7 @@ public void execute() throws ResourceUnavailableException, InsufficientCapacityE
         if (cluster == null) {
             throw new ServerApiException(ApiErrorCode.PARAM_ERROR, "Unable to find cluster by ID: " + getClusterId());
         }
-        final boolean result = haConfigManager.disableHA(cluster);
+        final boolean result = haConfigManager.disableHA(cluster, includeHost());
         CallContext.current().setEventDetails("Cluster ID:" + cluster.getUuid() + " HA enabled: false");
         CallContext.current().putContextParameter(Cluster.class, cluster.getUuid());
 

api/src/main/java/org/apache/cloudstack/api/command/admin/ha/EnableHAForClusterCmd.java

@@ -96,7 +96,7 @@ public void execute() throws ResourceUnavailableException, InsufficientCapacityE
             throw new ServerApiException(ApiErrorCode.PARAM_ERROR, "Unable to find cluster by ID: " + getClusterId());
         }
 
-        final boolean result = haConfigManager.enableHA(cluster);
+        final boolean result = haConfigManager.enableHA(cluster, includeHost());
         CallContext.current().setEventDetails("Cluster ID:" + cluster.getUuid() + " HA enabled: true");
         CallContext.current().putContextParameter(Cluster.class, cluster.getUuid());
 

api/src/main/java/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java

@@ -221,7 +221,7 @@ public String getStorageAccessGroup() {
     public ListHostsCmd(String storageAccessGroup) {
         this.storageAccessGroup = storageAccessGroup;
     }
-    
+
     public String getVersion() {
         return version;
     }

api/src/main/java/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java

@@ -33,9 +33,6 @@
 import org.apache.cloudstack.api.response.UserVmResponse;
 import org.apache.cloudstack.api.response.VolumeResponse;
 import org.apache.cloudstack.context.CallContext;
-import org.apache.commons.collections.CollectionUtils;
-import org.apache.commons.collections.MapUtils;
-import org.apache.commons.lang3.BooleanUtils;
 
 import com.cloud.exception.ConcurrentOperationException;
 import com.cloud.exception.InsufficientCapacityException;
@@ -44,9 +41,6 @@
 import com.cloud.exception.ResourceUnavailableException;
 import com.cloud.uservm.UserVm;
 import com.cloud.utils.exception.CloudRuntimeException;
-import com.cloud.utils.net.Dhcp;
-import com.cloud.utils.net.NetUtils;
-import com.cloud.utils.StringUtils;
 import com.cloud.vm.VirtualMachine;
 
 @APICommand(name = "deployVirtualMachine", description = "Creates and automatically starts an Instance based on a service offering, disk offering, and Template.", responseObject = UserVmResponse.class, responseView = ResponseView.Restricted, entityType = {VirtualMachine.class},
@@ -75,138 +69,6 @@ public class DeployVMCmd extends BaseDeployVMCmd {
     /////////////////// Accessors ///////////////////////
     /////////////////////////////////////////////////////
 
-    public String getAccountName() {
-        if (accountName == null) {
-            return CallContext.current().getCallingAccount().getAccountName();
-        }
-        return accountName;
-    }
-
-    public Long getDiskOfferingId() {
-        return diskOfferingId;
-    }
-
-    public String getDeploymentPlanner() {
-        return deploymentPlanner;
-    }
-
-    public String getDisplayName() {
-        if (StringUtils.isEmpty(displayName)) {
-            displayName = name;
-        }
-        return displayName;
-    }
-
-    public Long getDomainId() {
-        if (domainId == null) {
-            return CallContext.current().getCallingAccount().getDomainId();
-        }
-        return domainId;
-    }
-
-    public ApiConstants.BootType getBootType() {
-        if (StringUtils.isNotBlank(bootType)) {
-            try {
-                String type = bootType.trim().toUpperCase();
-                return ApiConstants.BootType.valueOf(type);
-            } catch (IllegalArgumentException e) {
-                String errMesg = "Invalid bootType " + bootType + "Specified for Instance " + getName()
-                        + " Valid values are: " + Arrays.toString(ApiConstants.BootType.values());
-                logger.warn(errMesg);
-                throw new InvalidParameterValueException(errMesg);
-            }
-        }
-        return null;
-    }
-
-    public ApiConstants.BootMode getBootMode() {
-        if (StringUtils.isNotBlank(bootMode)) {
-            try {
-                String mode = bootMode.trim().toUpperCase();
-                return ApiConstants.BootMode.valueOf(mode);
-            } catch (IllegalArgumentException e) {
-                String msg = String.format("Invalid %s: %s specified for Instance: %s. Valid values are: %s",
-                        ApiConstants.BOOT_MODE, bootMode, getName(), Arrays.toString(ApiConstants.BootMode.values()));
-                logger.error(msg);
-                throw new InvalidParameterValueException(msg);
-            }
-        }
-        if (ApiConstants.BootType.UEFI.equals(getBootType())) {
-            String msg = String.format("%s must be specified for the Instance with boot type: %s. Valid values are: %s",
-                    ApiConstants.BOOT_MODE, getBootType(), Arrays.toString(ApiConstants.BootMode.values()));
-            logger.error(msg);
-            throw new InvalidParameterValueException(msg);
-        }
-        return null;
-    }
-
-    public Map<String, String> getVmProperties() {
-        Map<String, String> map = new HashMap<>();
-        if (MapUtils.isNotEmpty(vAppProperties)) {
-            Collection parameterCollection = vAppProperties.values();
-            Iterator iterator = parameterCollection.iterator();
-            while (iterator.hasNext()) {
-                HashMap<String, String> entry = (HashMap<String, String>)iterator.next();
-                map.put(entry.get("key"), entry.get("value"));
-            }
-        }
-        return map;
-    }
-
-    public Map<Integer, Long> getVmNetworkMap() {
-        Map<Integer, Long> map = new HashMap<>();
-        if (MapUtils.isNotEmpty(vAppNetworks)) {
-            Collection parameterCollection = vAppNetworks.values();
-            Iterator iterator = parameterCollection.iterator();
-            while (iterator.hasNext()) {
-                HashMap<String, String> entry = (HashMap<String, String>) iterator.next();
-                Integer nic;
-                try {
-                    nic = Integer.valueOf(entry.get(VmDetailConstants.NIC));
-                } catch (NumberFormatException nfe) {
-                    nic = null;
-                }
-                String networkUuid = entry.get(VmDetailConstants.NETWORK);
-                if (logger.isTraceEnabled()) {
-                    logger.trace(String.format("nic, '%s', goes on net, '%s'", nic, networkUuid));
-                }
-                if (nic == null || StringUtils.isEmpty(networkUuid) || _entityMgr.findByUuid(Network.class, networkUuid) == null) {
-                    throw new InvalidParameterValueException(String.format("Network ID: %s for NIC ID: %s is invalid", networkUuid, nic));
-                }
-                map.put(nic, _entityMgr.findByUuid(Network.class, networkUuid).getId());
-            }
-        }
-        return map;
-    }
-
-    public String getGroup() {
-        return group;
-    }
-
-    public HypervisorType getHypervisor() {
-        return HypervisorType.getType(hypervisor);
-    }
-
-    public Boolean isDisplayVm() {
-        return displayVm;
-    }
-
-    @Override
-    public boolean isDisplay() {
-        if(displayVm == null)
-            return true;
-        else
-            return displayVm;
-    }
-
-    public List<String> getSecurityGroupNameList() {
-        return securityGroupNameList;
-    }
-
-    public List<Long> getSecurityGroupIdList() {
-        return securityGroupIdList;
-    }
-
     public Long getServiceOfferingId() {
         return serviceOfferingId;
     }

api/src/main/java/org/apache/cloudstack/api/command/user/vm/DeployVMVolumeCmd.java

@@ -754,7 +754,7 @@ public static String getResultObjectName() {
 
     @Override
     public long getEntityOwnerId() {
-        Long accountId = _accountService.finalyzeAccountId(accountName, domainId, projectId, true);
+        Long accountId = _accountService.finalizeAccountId(accountName, domainId, projectId, true);
         if (accountId == null) {
             return CallContext.current().getCallingAccount().getId();
         }

api/src/main/java/org/apache/cloudstack/api/command/user/vm/UpdateVMCmd.java

@@ -21,12 +21,7 @@
 import com.cloud.exception.ResourceUnavailableException;
 import com.cloud.user.Account;
 import com.cloud.uservm.UserVm;
-import java.util.Collection;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
 
-import com.cloud.utils.StringUtils;
 import com.cloud.utils.exception.CloudRuntimeException;
 import com.cloud.utils.net.Dhcp;
 import com.cloud.vm.VirtualMachine;

api/src/main/java/org/apache/cloudstack/api/response/LoginCmdResponse.java

@@ -97,7 +97,7 @@ public class LoginCmdResponse extends AuthenticationCmdResponse {
     @SerializedName(value = ApiConstants.EXTERNAL_ENTITY)
     @Param(description = "externalEntity")
     private String externalEntity;
-    
+
     @SerializedName(value = ApiConstants.PASSWORD_CHANGE_REQUIRED)
     @Param(description = "Indicates whether the User is required to change password on next login.", since = "4.23.0")
     private Boolean passwordChangeRequired;
@@ -251,7 +251,7 @@ public String getExternalEntity() {
     public void setExternalEntity(String externalEntity) {
         this.externalEntity = externalEntity;
     }
-    
+
     public Boolean getPasswordChangeRequired() {
         return passwordChangeRequired;
     }

api/src/main/java/org/apache/cloudstack/backup/BackupManager.java

@@ -252,7 +252,7 @@ public interface BackupManager extends BackupService, Configurable, PluggableSer
     Capacity getBackupStorageUsedStats(Long zoneId);
 
     void checkAndRemoveBackupOfferingBeforeExpunge(VirtualMachine vm);
-    
+
     static void validateBackupProviderConfig(String value) {
         if (value != null && (value.contains(",") || value.trim().contains(" "))) {
             throw new IllegalArgumentException("Multiple backup provider plugins are not supported. Please provide a single plugin value.");

core/src/main/java/org/apache/cloudstack/backup/RestoreBackupCommand.java

@@ -94,6 +94,14 @@ public void setRestoreVolumePaths(List<String> restoreVolumePaths) {
         this.restoreVolumePaths = restoreVolumePaths;
     }
 
+    public List<String> getVolumePaths() {
+        return volumePaths;
+    }
+
+    public void setVolumePaths(List<String> volumePaths) {
+        this.volumePaths = volumePaths;
+    }
+
     public List<String> getBackupFiles() {
         return backupFiles;
     }