[cloudstack] branch main updated: server: Fix migration issue in UserVmManagerImpl.migrateVirtualMachineWithVolume (#5288)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/main by this push:
     new 95de827  server: Fix migration issue in 
UserVmManagerImpl.migrateVirtualMachineWithVolume (#5288)
95de827 is described below

commit 95de82750edc626c90b9b305d4e92ad844fcbcc1
Author: Gabriel Beims Bräscher <gabr...@apache.org>
AuthorDate: Mon Aug 9 05:09:09 2021 -0300

    server: Fix migration issue in 
UserVmManagerImpl.migrateVirtualMachineWithVolume (#5288)
    
    I've upgraded a stage environment from an older 4.16-SNAPSHOT to the 
current one and found a regression bug at the VM migration.
    
    When calling the migrateVirtualMachineWithVolume, the following 
InvalidParameterValueException is launched: Unsupported hypervisor: KVM for VM 
migration, we support XenServer/VMware/KVM only.
---
 server/src/main/java/com/cloud/vm/UserVmManagerImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java 
b/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
index c56a826..4bd80da 100644
--- a/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
+++ b/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
@@ -6418,7 +6418,7 @@ public class UserVmManagerImpl extends ManagerBase 
implements UserVmManager, Vir
             throw new InvalidParameterValueException("Live Migration of GPU 
enabled VM is not supported");
         }
 
-        if 
(VM_STORAGE_MIGRATION_SUPPORTING_HYPERVISORS.contains(vm.getHypervisorType())) {
+        if 
(!VM_STORAGE_MIGRATION_SUPPORTING_HYPERVISORS.contains(vm.getHypervisorType())) 
{
             throw new 
InvalidParameterValueException(String.format("Unsupported hypervisor: %s for VM 
migration, we support XenServer/VMware/KVM only", vm.getHypervisorType()));
         }