Re: [Qemu-devel] [PATCH v4 00/13] tests: Introducing docker tests

["Alex Bennée" <alex.bennee@linaro.org>]

Fam Zheng <famz@redhat.com> writes:

> v4: Dropped the .gitignore patch in favor of tempfile [Alex];
>     Added one EXTRA_CONFIGURE_OPTS patch [Alex];
>
>     01: Fix commit message, and improve help text;
>         Fix pylint warnings, mostly long lines and some refactoring;
>         "--verbose" is now replaced with the shared args "--quiet";
>     02: Update commit message;
>         Use "--quiet", drop "--verbose";
>         Fix typo;
>     05: Mention "build_qemu" in commit message;
>         Add Alex's rev-by;
>     10: Fix stale commit message;
>
>     Add Alex's rev-by to v3 except above.
>
> This series adds a new "docker" make target family to run tests in created
> docker containers.
>
> To begin with, this can be a place to store standard env/command combinations to
> build and test QEMU.
>
> Secondly, CI usually provides "docker" capability, where we specify
> standard/repeatable test environments, and run tests in them.  However, what
> tests to cover is better maintained in-tree, in order to keep in sync with the
> code development.
>
> Lastly, this makes it very simple for developers to replicate such tests
> themselves.

What's the state of this series? Are we just waiting for the tree to
open for 2.7?

>
>
> Fam Zheng (13):
>   tests: Add utilities for docker testing
>   Makefile: Rules for docker testing
>   docker: Add images
>   docker: Add test runner
>   docker: Add common.rc
>   docker: Add quick test
>   docker: Add full test
>   docker: Add clang test
>   docker: Add mingw test
>   docker: Add travis tool
>   docs: Add text for tests/docker in build-system.txt
>   docker: Add EXTRA_CONFIGURE_OPTS
>   MAINTAINERS: Add tests/docker
>
>  MAINTAINERS                             |   7 ++
>  Makefile                                |   4 +-
>  docs/build-system.txt                   |   5 +
>  tests/docker/Makefile.include           | 124 +++++++++++++++++++++
>  tests/docker/common.rc                  |  32 ++++++
>  tests/docker/docker.py                  | 191 ++++++++++++++++++++++++++++++++
>  tests/docker/dockerfiles/centos6.docker |   6 +
>  tests/docker/dockerfiles/fedora.docker  |   7 ++
>  tests/docker/dockerfiles/ubuntu.docker  |  11 ++
>  tests/docker/run                        |  58 ++++++++++
>  tests/docker/test-clang                 |  25 +++++
>  tests/docker/test-full                  |  17 +++
>  tests/docker/test-mingw                 |  34 ++++++
>  tests/docker/test-quick                 |  19 ++++
>  tests/docker/travis                     |  21 ++++
>  tests/docker/travis.py                  |  48 ++++++++
>  16 files changed, 608 insertions(+), 1 deletion(-)
>  create mode 100644 tests/docker/Makefile.include
>  create mode 100755 tests/docker/common.rc
>  create mode 100755 tests/docker/docker.py
>  create mode 100644 tests/docker/dockerfiles/centos6.docker
>  create mode 100644 tests/docker/dockerfiles/fedora.docker
>  create mode 100644 tests/docker/dockerfiles/ubuntu.docker
>  create mode 100755 tests/docker/run
>  create mode 100755 tests/docker/test-clang
>  create mode 100755 tests/docker/test-full
>  create mode 100755 tests/docker/test-mingw
>  create mode 100755 tests/docker/test-quick
>  create mode 100755 tests/docker/travis
>  create mode 100755 tests/docker/travis.py


--
Alex Bennée