[libvirt] [PATCH 1/6] Remove virsh-all

Peter Krempa pkrempa at redhat.com
Mon Jun 20 10:23:30 UTC 2016


On Fri, Jun 17, 2016 at 20:04:36 +0200, Ján Tomko wrote:
> Since e8ac4a7 this test wastes some CPU cycles by blindly trying to
> run almost every virsh command, blindly throwing away the output
> and the return value and returning success if 'virsh help' successfully
> returned at least one command.
> 
> Drop it completely.
> ---
>  tests/Makefile.am |  1 -
>  tests/virsh-all   | 52 ----------------------------------------------------
>  2 files changed, 53 deletions(-)
>  delete mode 100755 tests/virsh-all

[...]

> diff --git a/tests/virsh-all b/tests/virsh-all
> deleted file mode 100755
> index 4a91e4e..0000000
> --- a/tests/virsh-all
> +++ /dev/null
> @@ -1,52 +0,0 @@


> -counter=0
> -for i in $cmds; do
> -    counter=`eval "expr $counter + 1"`
> -
> -    # For now, just run the command and ignore output
> -    $abs_top_builddir/tools/virsh -c $test_url $i < /dev/null > /dev/null 2>&1
> -    # Temporarily ignoring exit status
> -    #status=$?
> -    status=0

Lol!

> -    test_result $counter $i $status

ACK




More information about the libvir-list mailing list