sureshanaparti commented on code in PR #12970:
URL: https://github.com/apache/cloudstack/pull/12970#discussion_r3049348004


##########
api/src/main/java/org/apache/cloudstack/api/command/admin/vm/ImportVmCmd.java:
##########
@@ -179,6 +179,14 @@ public class ImportVmCmd extends 
ImportUnmanagedInstanceCmd {
             description = "(only for importing VMs from VMware to KVM) 
optional - the ID of the guest OS for the imported VM.")
     private Long guestOsId;
 
+    @Parameter(name = ApiConstants.USE_VDDK,
+            type = CommandType.BOOLEAN,
+            since = "4.22.1",
+            description = "(only for importing VMs from VMware to KVM) 
optional - if true, uses VDDK on the KVM conversion host for converting the VM. 
" +
+                    "This parameter is mutually exclusive with " + 
ApiConstants.FORCE_MS_TO_IMPORT_VM_FILES + ".")

Review Comment:
   can validate when both parameters are set and throw relevant error



-- 
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]

Reply via email to