[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 00/14] tests: Introducing docker tests
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v5 00/14] tests: Introducing docker tests |
Date: |
Fri, 27 May 2016 08:44:03 +0800 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Thu, 05/26 17:26, Alex Bennée wrote:
>
> Fam Zheng <address@hidden> writes:
>
> > v5: Use docker run's selinux option "z" on passed volume, drop --privileged.
> > Allow overriding "TARGET_LIST" in clang-test.
> > Add Alex's r-b lines in patches 1, 3, 9, 11 and 13.
> >
> > 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.
>
> Aside from the minor quibble on docker.py (which I can fix up on later
> patches if you want) I think this series it ready to be merged. Are you
> going to submit a PULL REQ to Peter?
Yes, I'll send one today.
Thanks,
Fam
>
> >
> >
> > Fam Zheng (14):
> > tests: Add utilities for docker testing
> > rules.mak: Add "COMMA" constant
> > 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 +
> > rules.mak | 2 +
> > 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 | 26 +++++
> > tests/docker/test-full | 17 +++
> > tests/docker/test-mingw | 34 ++++++
> > tests/docker/test-quick | 19 ++++
> > tests/docker/travis | 21 ++++
> > tests/docker/travis.py | 48 ++++++++
> > 17 files changed, 611 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
- [Qemu-devel] [PATCH v5 06/14] docker: Add common.rc, (continued)
- [Qemu-devel] [PATCH v5 06/14] docker: Add common.rc, Fam Zheng, 2016/05/23
- [Qemu-devel] [PATCH v5 09/14] docker: Add clang test, Fam Zheng, 2016/05/23
- [Qemu-devel] [PATCH v5 12/14] docs: Add text for tests/docker in build-system.txt, Fam Zheng, 2016/05/23
- [Qemu-devel] [PATCH v5 11/14] docker: Add travis tool, Fam Zheng, 2016/05/23
- [Qemu-devel] [PATCH v5 13/14] docker: Add EXTRA_CONFIGURE_OPTS, Fam Zheng, 2016/05/23
- [Qemu-devel] [PATCH v5 10/14] docker: Add mingw test, Fam Zheng, 2016/05/23
- [Qemu-devel] [PATCH v5 14/14] MAINTAINERS: Add tests/docker, Fam Zheng, 2016/05/23
- Re: [Qemu-devel] [PATCH v5 00/14] tests: Introducing docker tests, Alex Bennée, 2016/05/26
- Re: [Qemu-devel] [PATCH v5 00/14] tests: Introducing docker tests,
Fam Zheng <=