On Mon, 05/07 13:09, 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. > > Signed-off-by: Alistair Francis <alistair.fran...@wdc.com> > --- > 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 > ENV QEMU_CONFIGURE_OPTS --python=/usr/bin/python3 > +ENV PIP_PACKAGES \ > + avocado-framework avocado-qemu \
I think python-avacado overlaps with pip avacado-framework. Why are both needed? Fam > + avocado-framework-plugin-vt avocado-framework-plugin-runner-vm \ > + avocado-framework-plugin-runner-remote > avocado-framework-plugin-result-html > > 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 > +# > +# Copyright (c) 2018 Western Digital. > +# > +# Authors: > +# Alistair Francis <alistair.fran...@wdc.com> > +# > +# 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 docker-test-avocado@fedora > + > +. 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 > -- > 2.17.0 > >