[Libguestfs] [PATCH v3] v2v: -o vdsm: Add --vdsm-compat flag. -o rhev: Drop support for RHV < 4.1 (RHBZ#1400205).
Pino Toscano
ptoscano at redhat.com
Wed Dec 7 14:34:58 UTC 2016
On Wednesday, 7 December 2016 10:02:49 CET Richard W.M. Jones wrote:
> Support for RHEV with RHEL 6 nodes required us to output the old style
> qcow2 compat=0.10 images. Since RHEV 3.6 GA, RHEL 6 has not been
> supported as a RHEV node type. Since RHV 4.1, compat=1.1 is
> supported. (Support for compat=1.1 is uncertain in RHV 4.0 even on
> RHEL 7 nodes.)
>
> There are significant downsides to using qcow2 compat=0.10 instead of
> the modern default (compat=1.1).
>
> Therefore this patch does two things:
>
> For -o rhev, it drops support for compat=0.10 completely. You must
> use RHV 4.1.
>
> For -o vdsm, it adds an interim flag (--vdsm-compat=0.10 or
> --vdsm-compat=1.1) which controls the compat level of the qcow2 output
> file. VDSM should use --vdsm-compat=1.1 when it is known that modern
> qemu is available. We can make this the default later when all RHV
> instances have moved to 4.1.
>
> It also adds:
>
> vdsm-compat-option
>
> to the `virt-v2v --machine-readable' output to indicate that this flag
> can be used.
>
> Thanks: Yaniv Kaul, Michal Skrivanek.
> ---
LGTM, just one note below.
> diff --git a/v2v/test-v2v-o-vdsm-options.sh b/v2v/test-v2v-o-vdsm-options.sh
> index 3b7eaae..1de0e17 100755
> --- a/v2v/test-v2v-o-vdsm-options.sh
> +++ b/v2v/test-v2v-o-vdsm-options.sh
> @@ -21,6 +21,7 @@
> unset CDPATH
> export LANG=C
> set -e
> +set -x
This should be left out, I guess.
Thanks,
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20161207/27f1dbf1/attachment.sig>
More information about the Libguestfs
mailing list