slavkap commented on a change in pull request #5029:
URL: https://github.com/apache/cloudstack/pull/5029#discussion_r634488763
##########
File path: server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
##########
@@ -6208,8 +6208,8 @@ public VirtualMachine
migrateVirtualMachineWithVolume(Long vmId, Host destinatio
}
}
- if (!capabilities.isStorageMotionSupported()) {
- throw new CloudRuntimeException("Migration with storage isn't
supported on hypervisor " + srcHost.getHypervisorType() + " of version " +
srcHost.getHypervisorVersion());
+ if (capabilities == null || !capabilities.isStorageMotionSupported()) {
Review comment:
Thanks, @sureshanaparti, for the review! I tried to kill 2 birds with
one stone :)
I hope the new changes are better than this one
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
