search for: vddk_option_key

..._output_options output_options in > + `VDSM vdsm_options > + ) in Here I'd do the check of the options (both input and output) by prefixes, i.e. things like: List.iter ( fun key -> if not (String.is_prefix key "vddk-") || not (List.mem key vddk_option_keys) then error (f_"-it vddk: ‘-io %s’ is not a valid input option") key ) keys; So most probably adding a simple function to get the prefix of options per-input and per-output mode, and using it to check. The rest LGTM. -- Pino Toscano

...vdsm-image-uuid" ], Getopt.String ("uuid", set_output_option_compat "vdsm-image-uuid"), diff --git a/v2v/input_libvirt_vddk.ml b/v2v/input_libvirt_vddk.ml index f74ca05af..2ca0ef1e0 100644 --- a/v2v/input_libvirt_vddk.ml +++ b/v2v/input_libvirt_vddk.ml @@ -44,8 +44,7 @@ let vddk_option_keys = "port"; "snapshot"; "thumbprint"; - "transports"; - "vimapiver" ] + "transports" ] let print_input_options () = printf (f_"Input options (-io) which can be used with -it vddk: @@ -64,7 +63,6 @@ All oth...

...fchostport : string option; - vddk_port : string option; - vddk_snapshot : string option; - vddk_thumbprint : string option; - vddk_transports : string option; - vddk_vimapiver : string option; -} +type vddk_options = (string * string) list + +(* List of vddk-* input options. *) +let vddk_option_keys = + [ "vddk-config"; + "vddk-cookie"; + "vddk-libdir"; + "vddk-nfchostport"; + "vddk-port"; + "vddk-snapshot"; + "vddk-thumbprint"; + "vddk-transports"; + "vddk-vimapiver" ] + +let pr...

...string option; > - vddk_snapshot : string option; > - vddk_thumbprint : string option; > - vddk_transports : string option; > - vddk_vimapiver : string option; > -} > +type vddk_options = (string * string) list > + > +(* List of vddk-* input options. *) > +let vddk_option_keys = > + [ "vddk-config"; > + "vddk-cookie"; > + "vddk-libdir"; > + "vddk-nfchostport"; > + "vddk-port"; > + "vddk-snapshot"; > + "vddk-thumbprint"; > + "vddk-transports"; &gt...

This patch isn't quite complete (see ‘assert false’). Currently we have a bunch of ad hoc options like --vddk* and --vdsm* (and proposed to add --rhv*) to handle extra parameters for input and output modes/transports. This complicates the command line parsing and also the clarity of the command line (becauseit's not very obvious which options apply to which side of the conversion).

v6 was here: https://www.redhat.com/archives/libguestfs/2018-March/msg00126.html This makes a number of significant changes: - Input and output options now use a uniform set of -io and -oo parameters. - For -o rhv-upload, we use ‘-oo rhv-cafile=/tmp/ca.pem’ etc. The ‘--rhv*’ options have been dropped. - Rearranges the documentation. - As before includes (untested) support for zero,