<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Nov 25, 2019, 16:37 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 Mon, Nov 25, 2019 at 04:27:11PM +0200, Nir Soffer wrote:<br>
> On Mon, Nov 25, 2019, 11:15 Richard W.M. Jones <<a href="mailto:rjones@redhat.com" target="_blank" rel="noreferrer">rjones@redhat.com</a>> wrote:<br>
> <br>
> > 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<br>
> > target_format<br>
> > > +        * which is the disk format. commpressed format will also not<br>
> > 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>
> ><br>
> <br>
> Maybe outpu#tranfer_format?<br>
> Return output_format by default, rhv_output will override it to return raw?<br>
<br>
Isn't that essentially what #override_output_format virtual method<br>
does already?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">The name confused me, I will check this then.</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>
virt-top is 'top' for virtual machines.  Tiny program with many<br>
powerful monitoring features, net stats, disk stats, logging, etc.<br>
<a href="http://people.redhat.com/~rjones/virt-top" rel="noreferrer noreferrer" target="_blank">http://people.redhat.com/~rjones/virt-top</a><br>
<br>
</blockquote></div></div></div>