[Libguestfs] [nbdkit PATCH] linuxdisk: Reduce size of test

[Richard W.M. Jones]

On Tue, May 31, 2022 at 04:10:07PM -0500, Eric Blake wrote:
> Using all of ../plugins as the contents for the linuxdisk gets
> progressively bigger over time with incremental builds.  Before I
> cleaned it up, my plugins/rust/target/debug had accumulated over 6G of
> cruft, causing test-linuxdisk-copy-out.sh to fail due to ENOSPC during
> mke2fs.  Pick a smaller directory tree to export, but still test that
> we expose a subdirectory.
> 
> Fixes: 5da0f2fc9 ("Add linuxdisk plugin.")
> ---
>  plugins/linuxdisk/subdir/.gitignore |  0
>  tests/test-linuxdisk-copy-out.sh    | 18 +++++++++---------
>  2 files changed, 9 insertions(+), 9 deletions(-)
>  create mode 100644 plugins/linuxdisk/subdir/.gitignore
> 
> diff --git a/plugins/linuxdisk/subdir/.gitignore b/plugins/linuxdisk/subdir/.gitignore
> new file mode 100644
> index 00000000..e69de29b
> diff --git a/tests/test-linuxdisk-copy-out.sh b/tests/test-linuxdisk-copy-out.sh
> index 3a38fb58..f926a55c 100755
> --- a/tests/test-linuxdisk-copy-out.sh
> +++ b/tests/test-linuxdisk-copy-out.sh
> @@ -1,6 +1,6 @@
>  #!/usr/bin/env bash
>  # nbdkit
> -# Copyright (C) 2018-2020 Red Hat Inc.
> +# Copyright (C) 2018-2022 Red Hat Inc.
>  #
>  # Redistribution and use in source and binary forms, with or without
>  # modification, are permitted provided that the following conditions are
> @@ -49,25 +49,25 @@ cleanup_fn rm -f $files
> 
>  nbdkit -f -v -U - \
>         --filter=partition \
> -       linuxdisk $srcdir/../plugins partition=1 label=ROOT \
> +       linuxdisk $srcdir/../plugins/linuxdisk partition=1 label=ROOT \
>         --run 'nbdcopy "$uri" linuxdisk-copy-out.img'
> 
>  # Check the disk content.
>  guestfish --ro -a linuxdisk-copy-out.img -m /dev/sda <<EOF
>  # Check some known files and directories exist.
>    ll /
> -  ll /linuxdisk
> -  is-dir /linuxdisk
> -  is-file /linuxdisk/Makefile.am
> +  ll /subdir
> +  is-dir /subdir
> +  is-file /Makefile.am
> 
>  # This reads out all the directory entries and all file contents.
>    tar-out / - | cat >/dev/null
> 
>  # Download some files and compare to local copies.
> -  download /linuxdisk/Makefile linuxdisk-copy-out.test1
> -  download /linuxdisk/Makefile.am linuxdisk-copy-out.test2
> -  download /linuxdisk/nbdkit-linuxdisk-plugin.pod linuxdisk-copy-out.test3
> -  download /linuxdisk/filesystem.c linuxdisk-copy-out.test4
> +  download /Makefile linuxdisk-copy-out.test1
> +  download /Makefile.am linuxdisk-copy-out.test2
> +  download /nbdkit-linuxdisk-plugin.pod linuxdisk-copy-out.test3
> +  download /filesystem.c linuxdisk-copy-out.test4
>  EOF
> 
>  # Compare downloaded files to local versions.
> -- 

Looks good, so:

Reviewed-by: Richard W.M. Jones <rjones at redhat.com>

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-top is 'top' for virtual machines.  Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://people.redhat.com/~rjones/virt-top