[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 1/1] tests/docker: Add a Avocado Docker test
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v1 1/1] tests/docker: Add a Avocado Docker test |
Date: |
Mon, 7 May 2018 19:44:50 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
Hi Alistair,
On 05/07/2018 05:09 PM, Alistair Francis wrote:
> Avocado is not trivial to setup on non-Fedora systems. To simplfying
> future testing add a docker test image that runs Avocado tests.
How big is the image with the new packages?
If not everybody is interested in using Avocado and Python 3 as default.
You might use the fedora as a base image for a 'fedora-avocado' one via
'FROM qemu:fedora' and adding a 'docker-image-fedora-avocado:
docker-image-fedora' dependency in Makefile.include.
>
> Signed-off-by: Alistair Francis <address@hidden>
> ---
> tests/docker/dockerfiles/fedora.docker | 12 +++++++++--
> tests/docker/test-avocado | 28 ++++++++++++++++++++++++++
> 2 files changed, 38 insertions(+), 2 deletions(-)
> create mode 100755 tests/docker/test-avocado
>
> diff --git a/tests/docker/dockerfiles/fedora.docker
> b/tests/docker/dockerfiles/fedora.docker
> index b706f42405..5c23f6e459 100644
> --- a/tests/docker/dockerfiles/fedora.docker
> +++ b/tests/docker/dockerfiles/fedora.docker
> @@ -18,9 +18,17 @@ ENV PACKAGES \
> mingw64-pixman mingw64-glib2 mingw64-gmp mingw64-SDL mingw64-pkg-config \
> mingw64-gtk2 mingw64-gtk3 mingw64-gnutls mingw64-nettle mingw64-libtasn1
> \
> mingw64-libjpeg-turbo mingw64-libpng mingw64-curl mingw64-libssh2 \
> - mingw64-bzip2
> + mingw64-bzip2 \
> + xz tcpdump nc \
> + python2-devel python3-devel python-avocado \
> + libvirt-devel qemu-kvm
I'd personally prefer a commit that sort this list alphabetically (one
package per line), and another commit only adding the Avocado packages;
but this is up to Fam.
> ENV QEMU_CONFIGURE_OPTS --python=/usr/bin/python3
> +ENV PIP_PACKAGES \
> + avocado-framework avocado-qemu \
> + avocado-framework-plugin-vt avocado-framework-plugin-runner-vm \
> + avocado-framework-plugin-runner-remote
> avocado-framework-plugin-result-html
Ditto, alphabetically sorted one per line.
>
> RUN dnf install -y $PACKAGES
> +RUN pip install $PIP_PACKAGES
> RUN rpm -q $PACKAGES | sort > /packages.txt
> -ENV FEATURES mingw clang pyyaml asan
> +ENV FEATURES mingw clang pyyaml asan avocado
> diff --git a/tests/docker/test-avocado b/tests/docker/test-avocado
> new file mode 100755
> index 0000000000..8da7bc10a2
> --- /dev/null
> +++ b/tests/docker/test-avocado
> @@ -0,0 +1,28 @@
> +#!/bin/bash -e
> +#
> +# Avocado tests on Fedora, as these are a real pain on Debian systems
Have been here... Then used the git version :/
> +#
> +# Copyright (c) 2018 Western Digital.
> +#
> +# Authors:
> +# Alistair Francis <address@hidden>
> +#
> +# This work is licensed under the terms of the GNU GPL, version 2
> +# or (at your option) any later version. See the COPYING file in
> +# the top-level directory.
> +#
> +# Run this test: NETWORK=1 make address@hidden
> +
> +. common.rc
> +
> +requires avocado
> +
> +cd "$BUILD_DIR"
> +
> +DEF_TARGET_LIST="x86_64-softmmu"
> +TARGET_LIST=${TARGET_LIST:-$DEF_TARGET_LIST} \
> +build_qemu
> +install_qemu
> +export PATH="${PATH}:$(pwd)"
> +avocado vt-bootstrap --yes-to-all --vt-type qemu
> +avocado run boot --vt-qemu-bin ./x86_64-softmmu/qemu-system-x86_64
So this script is for the VT version, specific to x86. I'd rather name
the script accordingly "test-avocado-vt".
Regards,
Phil.