[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 02/13] Makefile: Rules for docker testing
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v4 02/13] Makefile: Rules for docker testing |
Date: |
Thu, 31 Mar 2016 10:52:47 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.0.92.2 |
Fam Zheng <address@hidden> writes:
> This adds a group of make targets to run docker tests, all are available
> in source tree without running ./configure.
>
> The usage is shown with "make docker".
>
> Besides the fixed ones, dynamic targets for building each image and
> running each test in each image are generated automatically by make,
> scanning $(SRC_PATH)/tests/docker/ files with specific patterns.
>
> Alternative to manually list particular targets (address@hidden)
> set, you can control which tests/images to run by filtering variables,
> TESTS= and IMAGES=, which are expressed in Makefile pattern syntax,
> "foo% %bar ...". For example:
>
> $ make docker-test IMAGES="ubuntu fedora"
$ [qemu.git/review/docker-v4] >make docker-test IMAGES="fedora" DEBUG=1
V=1
/home/alex/lsrc/qemu/qemu.git/rules.mak:178: warning: overriding recipe for
target 'clean-timestamp'
/home/alex/lsrc/qemu/qemu.git/rules.mak:178: warning: ignoring old recipe for
target 'clean-timestamp'
/home/alex/lsrc/qemu/qemu.git/tests/docker/docker.py build qemu:fedora
/home/alex/lsrc/qemu/qemu.git/tests/docker/dockerfiles/fedora.docker
Image is up to date.
(cd /home/alex/lsrc/qemu/qemu.git; if git diff-index --quiet HEAD --
&>/dev/null; then git archive -1 HEAD --format=tar.gz -o
/tmp/qemu-src.opTOn/qemu.tgz; else git archive -1 $(git stash create)
--format=tar.gz -o /tmp/qemu-src.opTOn/qemu.tgz; fi)
cp "/home/alex/lsrc/qemu/qemu.git/tests/docker/run"
"/tmp/qemu-src.opTOn/run"
/home/alex/lsrc/qemu/qemu.git/tests/docker/docker.py run --privileged
-t -i -e TARGET_LIST= -e EXTRA_CONFIGURE_OPTS= -e V=1 -e J= -e DEBUG=1
-e CCACHE_DIR=/var/tmp/ccache -v $(realpath
/tmp/qemu-src.opTOn):/var/tmp/qemu:ro -v
/var/tmp/qemu-docker-ccache:/var/tmp/ccache -w /var/tmp/qemu qemu:fedora
/bin/bash -x ./run test-clang;
docker: Error response from daemon: Container command not found or does not
exist..
/home/alex/lsrc/qemu/qemu.git/tests/docker/Makefile.include:102: recipe for
target 'address@hidden' failed
make: *** address@hidden Error 127
I can't get a test shell on the container, but I suspect that means
./run isn't in the path somewhere?
>
> Unfortunately, it's impossible to propagate "-j $JOBS" into make in
> containers, however since each combination is made a first class target
> in the top Makefile, "make -j$N docker-test" still parallels the tests
> coarsely.
>
> Still, $J is made a magic variable to let all make invocations in
> containers to use -j$J.
>
> Instead of providing a live version of the source tree to the docker
> container we snapshot it with git-archive. This ensures the tree is in a
> pristine state for whatever operations the container is going to run on
> them.
>
> Uncommitted changes known to files known by the git index will be
> included in the snapshot if there are any.
>
> Reviewed-by: Alex Bennée <address@hidden>
> Signed-off-by: Fam Zheng <address@hidden>
> Signed-off-by: Alex Bennée <address@hidden>
> ---
> Makefile | 4 +-
> tests/docker/Makefile.include | 121
> ++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 124 insertions(+), 1 deletion(-)
> create mode 100644 tests/docker/Makefile.include
>
> diff --git a/Makefile b/Makefile
> index 1d076a9..554a3b1 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -6,7 +6,7 @@ BUILD_DIR=$(CURDIR)
> # Before including a proper config-host.mak, assume we are in the source tree
> SRC_PATH=.
>
> -UNCHECKED_GOALS := %clean TAGS cscope ctags
> +UNCHECKED_GOALS := %clean TAGS cscope ctags docker docker-%
>
> # All following code might depend on configuration variables
> ifneq ($(wildcard config-host.mak),)
> @@ -651,3 +651,5 @@ endif
> # Include automatically generated dependency files
> # Dependencies in Makefile.objs files come from our recursive subdir rules
> -include $(wildcard *.d tests/*.d)
> +
> +include $(SRC_PATH)/tests/docker/Makefile.include
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> new file mode 100644
> index 0000000..033809a
> --- /dev/null
> +++ b/tests/docker/Makefile.include
> @@ -0,0 +1,121 @@
> +# Makefile for Docker tests
> +
> +$(if $(quiet-command),,$(eval include $(SRC_PATH)/rules.mak))
> +
> +.PHONY: docker docker-test docker-clean docker-image docker-qemu-src
> +
> +DOCKER_SUFFIX := .docker
> +DOCKER_FILES_DIR := $(SRC_PATH)/tests/docker/dockerfiles
> +DOCKER_IMAGES := $(notdir $(basename $(wildcard
> $(DOCKER_FILES_DIR)/*.docker)))
> +DOCKER_TARGETS := $(patsubst %,docker-image-%,$(DOCKER_IMAGES))
> +# Use a global constant ccache directory to speed up repetitive builds
> +DOCKER_CCACHE_DIR := /var/tmp/qemu-docker-ccache
> +
> +DOCKER_TESTS := $(notdir $(shell \
> + find $(SRC_PATH)/tests/docker/ -name 'test-*' -type f -executable))
> +
> +DOCKER_TOOLS := travis
> +
> +TESTS ?= %
> +IMAGES ?= %
> +SRC_COPY := $(shell mktemp -u /tmp/qemu-src.XXXXX)
> +
> +# Make archive from git repo $1 to tar.gz $2
> +make-archive-maybe = $(if $(wildcard $1/*), \
> + $(call quiet-command, \
> + (cd $1; if git diff-index --quiet HEAD -- &>/dev/null; then \
> + git archive -1 HEAD --format=tar.gz -o $2; \
> + else \
> + git archive -1 $$(git stash create) --format=tar.gz -o
> $2; \
> + fi), \
> + " ARCHIVE $(notdir $2)"))
> +
> +$(SRC_COPY):
> + @mkdir -p $@
> + $(call make-archive-maybe, $(SRC_PATH), $(SRC_COPY)/qemu.tgz)
> + $(call make-archive-maybe, $(SRC_PATH)/dtc, $(SRC_COPY)/dtc.tgz)
> + $(call make-archive-maybe, $(SRC_PATH)/pixman, $(SRC_COPY)/pixman.tgz)
> + $(call quiet-command, cp "$(SRC_PATH)/tests/docker/run"
> "$(SRC_COPY)/run", \
> + " COPY RUNNER")
> +
> +docker-qemu-src: $(SRC_COPY)
> +
> +docker-image: ${DOCKER_TARGETS}
> +
> +# General rule for building docker images
> +docker-image-%: $(DOCKER_FILES_DIR)/%.docker
> + $(call quiet-command,\
> + $(SRC_PATH)/tests/docker/docker.py build qemu:$* $< \
> + $(if $V,,--quiet) $(if $(NOCACHE),--no-cache),\
> + " BUILD $*")
> +
> +# Expand all the pre-requistes for each docker image and test combination
> +$(foreach i,$(DOCKER_IMAGES), \
> + $(foreach t,$(DOCKER_TESTS) $(DOCKER_TOOLS), \
> + $(eval .PHONY: address@hidden) \
> + $(eval address@hidden: docker-image-$i address@hidden) \
> + ) \
> + $(foreach t,$(DOCKER_TESTS), \
> + $(eval docker-test: address@hidden) \
> + ) \
> +)
> +
> +docker:
> + @echo 'Build QEMU and run tests inside Docker containers'
> + @echo
> + @echo 'Available targets:'
> + @echo
> + @echo ' docker: Print this help.'
> + @echo ' docker-test: Run all image/test combinations.'
> + @echo ' docker-clean: Kill and remove residual docker testing
> containers.'
> + @echo ' address@hidden: Run "TEST" in container "IMAGE".'
> + @echo ' Note: "TEST" is one of the listed test
> name,'
> + @echo ' or a script name under
> $$QEMU_SRC/tests/docker/;'
> + @echo ' "IMAGE" is one of the listed container
> name."'
> + @echo ' docker-image: Build all images.'
> + @echo ' docker-image-IMAGE: Build image "IMAGE".'
> + @echo
> + @echo 'Available container images:'
> + @echo ' $(DOCKER_IMAGES)'
> + @echo
> + @echo 'Available tests:'
> + @echo ' $(DOCKER_TESTS)'
> + @echo
> + @echo 'Available tools:'
> + @echo ' $(DOCKER_TOOLS)'
> + @echo
> + @echo 'Special variables:'
> + @echo ' TARGET_LIST=a,b,c Override target list in builds.'
> + @echo ' IMAGES="a b c ..": Filters which images to build or run.'
> + @echo ' TESTS="x y z .." Filters which tests to run (for
> docker-test).'
> + @echo ' J=[0..9]* Overrides the -jN parameter for make
> commands'
> + @echo ' (default is 1)'
> + @echo ' DEBUG=1 Stop and drop to shell in the created
> container'
> + @echo ' before running the command.'
> + @echo ' NOCACHE=1 Ignore cache when build images.'
> +
> +docker-run-%: CMD = $(shell echo '$@' | sed -e
> 's/docker-run-\(address@hidden)@\(.*\)/\1/')
> +docker-run-%: IMAGE = $(shell echo '$@' | sed -e
> 's/docker-run-\(address@hidden)@\(.*\)/\2/')
> +docker-run-%: docker-qemu-src
> + @if test -z "$(IMAGE)" || test -z "$(CMD)"; \
> + then echo "Invalid target"; exit 1; \
> + fi
> + $(if $(filter $(TESTS),$(CMD)),$(if $(filter $(IMAGES),$(IMAGE)), \
> + $(call quiet-command,\
> + $(SRC_PATH)/tests/docker/docker.py run $(if $V,,--rm) \
> + --privileged -t \
> + $(if $(DEBUG),-i,--net=none) \
> + -e TARGET_LIST=$(TARGET_LIST) \
> + -e V=$V -e J=$J -e DEBUG=$(DEBUG)\
> + -e CCACHE_DIR=/var/tmp/ccache \
> + -v $$(realpath $(SRC_COPY)):/var/tmp/qemu:ro \
> + -v $(DOCKER_CCACHE_DIR):/var/tmp/ccache \
> + -w /var/tmp/qemu \
> + qemu:$(IMAGE) \
> + $(if $V,/bin/bash -x ,) \
> + ./run \
> + $(CMD); \
> + , " RUN $(CMD) in $(IMAGE)")))
> +
> +docker-clean:
> + $(call quiet-command, $(SRC_PATH)/tests/docker/docker.py clean)
--
Alex Bennée
- [Qemu-devel] [PATCH v4 00/13] tests: Introducing docker tests, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 01/13] tests: Add utilities for docker testing, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 02/13] Makefile: Rules for docker testing, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 04/13] docker: Add test runner, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 03/13] docker: Add images, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 05/13] docker: Add common.rc, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 06/13] docker: Add quick test, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 07/13] docker: Add full test, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 08/13] docker: Add clang test, Fam Zheng, 2016/03/17
- [Qemu-devel] [PATCH v4 09/13] docker: Add mingw test, Fam Zheng, 2016/03/17