<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Nov 25, 2019, 11:15 Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Nov 20, 2019 at 03:06:55AM +0200, Nir Soffer wrote:<br>
> diff --git a/v2v/<a href="http://v2v.ml" rel="noreferrer noreferrer" target="_blank">v2v.ml</a> b/v2v/<a href="http://v2v.ml" rel="noreferrer noreferrer" target="_blank">v2v.ml</a><br>
> index 03590c9e..58bb06c3 100644<br>
> --- a/v2v/<a href="http://v2v.ml" rel="noreferrer noreferrer" target="_blank">v2v.ml</a><br>
> +++ b/v2v/<a href="http://v2v.ml" rel="noreferrer noreferrer" target="_blank">v2v.ml</a><br>
> @@ -739,7 +739,9 @@ and copy_targets cmdline targets input output =<br>
>            | TargetURI uri -> uri in<br>
>          [ "qemu-img"; "convert" ] @<br>
>          (if not (quiet ()) then [ "-p" ] else []) @<br>
> -        [ "-n"; "-f"; "qcow2"; "-O"; t.target_format ] @<br>
> +        (* XXX When using NBD we must use raw format, not the target_format<br>
> +        * which is the disk format. commpressed format will also not work. *)<br>
> +        [ "-n"; "-f"; "qcow2"; "-O"; "raw" ] @<br>
<br>
This is going to break all the other output modes.<br>
<br>
You probably want to define a new method output#override_output_format<br>
(see v2v.ml:get_target_formats and types.mli).<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Maybe outpu#tranfer_format?</div><div dir="auto"><br></div><div dir="auto">Return output_format by default, rhv_output will override it to return raw?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
Fedora Windows cross-compiler. Compile Windows programs, test, and<br>
build Windows installers. Over 100 libraries supported.<br>
<a href="http://fedoraproject.org/wiki/MinGW" rel="noreferrer noreferrer" target="_blank">http://fedoraproject.org/wiki/MinGW</a><br>
<br>
</blockquote></div></div></div>