[Libguestfs] [nbdkit PATCH] tests: Accommodate qemu-img 4.1 output change
Richard W.M. Jones
rjones at redhat.com
Tue Jul 30 21:06:57 UTC 2019
On Tue, Jul 30, 2019 at 04:04:14PM -0500, Eric Blake wrote:
> Where qemu-img 4.0 used to say 'virtual size: 100M', the 4.1 release
> now says 'virtual size: 100 MiB'. Similarly, '5.0G' turned into '5
> GiB'. Update expected test output to tolerate either version of qemu.
This patch is fine and less invasive than I feared. I don't know if
you want to do a second version using ‘--output=json’ and ‘jq’ to make
it more future-proof.
ACK
Rich.
> Signed-off-by: Eric Blake <eblake at redhat.com>
> ---
>
> I already know I want to send a v2; on IRC, Rich pointed out that
> 'qemu-img info --output=json' is less prone to fickle changes, and we
> already depend on jq elsewhere in the testsuite. But since I'd
> already got this written up, I'm at least posting it (if nothing else,
> to have a list archive to point to when someone else complains about
> qemu-img changing output).
>
> tests/test-ip.sh | 4 ++--
> tests/test-nbd-tls-psk.sh | 2 +-
> tests/test-nbd-tls.sh | 2 +-
> tests/test-tls-psk.sh | 2 +-
> tests/test-tls.sh | 2 +-
> tests/test-truncate3.sh | 2 +-
> 6 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/tests/test-ip.sh b/tests/test-ip.sh
> index 636d3d3f..60f2e066 100755
> --- a/tests/test-ip.sh
> +++ b/tests/test-ip.sh
> @@ -59,7 +59,7 @@ ipv4_lo="$(ip -o -4 addr show scope host)"
> if test -n "$ipv4_lo"; then
> qemu-img info --image-opts "file.driver=nbd,file.host=127.0.0.1,file.port=$port" > ipv4.out
> cat ipv4.out
> - grep -sq "^virtual size: 100M" ipv4.out
> + grep -sq "^virtual size: 100 *M" ipv4.out
> fi
>
> # Check we can connect over the IPv6 loopback interface.
> @@ -67,5 +67,5 @@ ipv6_lo="$(ip -o -6 addr show scope host)"
> if test -n "$ipv6_lo"; then
> qemu-img info --image-opts "file.driver=nbd,file.host=::1,file.port=$port" > ipv6.out
> cat ipv6.out
> - grep -sq "^virtual size: 100M" ipv6.out
> + grep -sq "^virtual size: 100 *M" ipv6.out
> fi
> diff --git a/tests/test-nbd-tls-psk.sh b/tests/test-nbd-tls-psk.sh
> index d0bbc468..e07d553b 100755
> --- a/tests/test-nbd-tls-psk.sh
> +++ b/tests/test-nbd-tls-psk.sh
> @@ -78,4 +78,4 @@ LANG=C qemu-img info -f raw "nbd+unix:///?socket=$sock2" > nbd-tls-psk.out
> cat nbd-tls-psk.out
>
> grep -sq "^file format: raw" nbd-tls-psk.out
> -grep -sq "^virtual size: 100M" nbd-tls-psk.out
> +grep -sq "^virtual size: 100 *M" nbd-tls-psk.out
> diff --git a/tests/test-nbd-tls.sh b/tests/test-nbd-tls.sh
> index af824d23..11fdea22 100755
> --- a/tests/test-nbd-tls.sh
> +++ b/tests/test-nbd-tls.sh
> @@ -79,4 +79,4 @@ LANG=C qemu-img info -f raw "nbd+unix:///?socket=$sock2" > nbd-tls.out
> cat nbd-tls.out
>
> grep -sq "^file format: raw" nbd-tls.out
> -grep -sq "^virtual size: 100M" nbd-tls.out
> +grep -sq "^virtual size: 100 *M" nbd-tls.out
> diff --git a/tests/test-tls-psk.sh b/tests/test-tls-psk.sh
> index 393f5893..c0f03487 100755
> --- a/tests/test-tls-psk.sh
> +++ b/tests/test-tls-psk.sh
> @@ -80,4 +80,4 @@ qemu-img info \
> cat tls-psk.out
>
> grep -sq "^file format: raw" tls-psk.out
> -grep -sq "^virtual size: 100M" tls-psk.out
> +grep -sq "^virtual size: 100 *M" tls-psk.out
> diff --git a/tests/test-tls.sh b/tests/test-tls.sh
> index 70d40aea..5936f9fa 100755
> --- a/tests/test-tls.sh
> +++ b/tests/test-tls.sh
> @@ -73,4 +73,4 @@ qemu-img info \
> cat tls.out
>
> grep -sq "^file format: raw" tls.out
> -grep -sq "^virtual size: 100M" tls.out
> +grep -sq "^virtual size: 100 *M" tls.out
> diff --git a/tests/test-truncate3.sh b/tests/test-truncate3.sh
> index 0a7fba8b..396fd253 100755
> --- a/tests/test-truncate3.sh
> +++ b/tests/test-truncate3.sh
> @@ -51,7 +51,7 @@ start_nbdkit -P truncate3.pid -U $sock \
> round-up=512
>
> LANG=C qemu-img info nbd:unix:$sock > truncate3.out
> -if ! grep "virtual size: 5.0G" truncate3.out; then
> +if ! grep "virtual size:.*(5368709120 " truncate3.out; then
> echo "$0: unexpected output from truncate3 regression test:"
> cat truncate3.out
> exit 1
> --
> 2.20.1
>
> _______________________________________________
> Libguestfs mailing list
> Libguestfs at redhat.com
> https://www.redhat.com/mailman/listinfo/libguestfs
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines. Supports shell scripting,
bindings from many languages. http://libguestfs.org
More information about the Libguestfs
mailing list