sureshanaparti commented on code in PR #12970:
URL: https://github.com/apache/cloudstack/pull/12970#discussion_r3055521882
##########
server/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManagerImpl.java:
##########
@@ -1725,12 +1743,11 @@ protected UserVm
importUnmanagedInstanceFromVmwareToKvm(DataCenter zone, Cluster
serviceOffering, dataDiskOfferingMap,
temporaryConvertLocation,
ovfTemplateOnConvertLocation, forceConvertToPool,
extraParams);
} else {
- // Uses KVM Host for OVF export to temporary conversion
location, through ovftool
- importVmTasksManager.updateImportVMTaskStep(importVMTask,
zone, owner, convertHost, importHost, null, ConvertingInstance);
- convertedInstance =
convertVmwareInstanceToKVMAfterExportingOVFToConvertLocation(
+ // Uses KVM Host for direct conversion using VDDK, or for OVF
export to temporary conversion location through ovftool
importVmTasksManager.updateImportVMTaskStep(importVMTask, zone, owner,
convertHost, importHost, null, ConvertingInstance);
Review Comment:
```suggestion
// Uses KVM Host for direct conversion using VDDK, or for
OVF export to temporary conversion location through ovftool
importVmTasksManager.updateImportVMTaskStep(importVMTask,
zone, owner, convertHost, importHost, null, ConvertingInstance);
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
