From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34239) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b65sq-0002xn-Ha for qemu-devel@nongnu.org; Thu, 26 May 2016 20:44:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b65sk-0000cl-9O for qemu-devel@nongnu.org; Thu, 26 May 2016 20:44:11 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42240) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b65sk-0000cO-0n for qemu-devel@nongnu.org; Thu, 26 May 2016 20:44:06 -0400 Date: Fri, 27 May 2016 08:44:03 +0800 From: Fam Zheng Message-ID: <20160527004403.GI31052@ad.usersys.redhat.com> References: <1463986466-764-1-git-send-email-famz@redhat.com> <87eg8odbu4.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <87eg8odbu4.fsf@linaro.org> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v5 00/14] tests: Introducing docker tests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex =?iso-8859-1?Q?Benn=E9e?= Cc: qemu-devel@nongnu.org, Peter Maydell On Thu, 05/26 17:26, Alex Benn=E9e wrote: >=20 > Fam Zheng writes: >=20 > > v5: Use docker run's selinux option "z" on passed volume, drop --priv= ileged. > > 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 cr= eated > > docker containers. > > > > To begin with, this can be a place to store standard env/command comb= inations to > > build and test QEMU. > > > > Secondly, CI usually provides "docker" capability, where we specify > > standard/repeatable test environments, and run tests in them. Howeve= r, 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 te= sts > > themselves. >=20 > 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 >=20 > > > > > > 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 >=20 >=20 > -- > Alex Benn=E9e