[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 02/15] Makefile: Rules for docker testing
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v2 02/15] Makefile: Rules for docker testing |
Date: |
Tue, 1 Mar 2016 09:04:48 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, 02/29 17:08, Alex Bennée wrote:
>
> 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 by "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-run IMAGES="ubuntu fedora"
>
> This doesn't seem to work for me:
>
> 16:49 address@hidden/x86_64 [qemu.git/review/docker-tests-v2] >make
> docker-run IMAGES="ubuntu" V=1
> /home/alex/lsrc/qemu/qemu.git/rules.mak:178: warning: overriding
> commands for target `clean-timestamp'
> /home/alex/lsrc/qemu/qemu.git/rules.mak:178: warning: ignoring old
> commands for target `clean-timestamp'
> make: *** No rule to make target `docker-run'. Stop.
The commit message is stale, as the help text, the target is now "docker-test"
which is more intuitive.
Fam
> `
>
> >
> > Unfortunately, it's impossible to propagate "-j $JOBS" into make in
> > containers, however since each combination is made a first class target
> > is the top Makefile, "make -j$N docker-run" still parallels the tests
> > coarsely.
> >
> > Signed-off-by: Fam Zheng <address@hidden>
> > ---
> > Makefile | 4 +-
> > tests/docker/Makefile.include | 93
> > +++++++++++++++++++++++++++++++++++++++++++
> > 2 files changed, 96 insertions(+), 1 deletion(-)
> > create mode 100644 tests/docker/Makefile.include
> >
> > diff --git a/Makefile b/Makefile
> > index f9fae3a..f104a08 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..d8c55c3
> > --- /dev/null
> > +++ b/tests/docker/Makefile.include
> > @@ -0,0 +1,93 @@
> > +# Makefile for Docker tests
> > +
> > +$(if $(quiet-command),,$(eval include $(SRC_PATH)/rules.mak))
> > +
> > +.PHONY: docker docker-test docker-clean docker-image
> > +
> > +DOCKER_SUFFIX := .docker
> > +
> > +DOCKER_IMAGES := $(notdir $(basename \
> > + $(wildcard $(SRC_PATH)/tests/docker/dockerfiles/*)))
> > +
> > +DOCKER_TESTS := $(notdir $(shell \
> > + find $(SRC_PATH)/tests/docker/ -name 'test-*' -type f -executable))
> > +
> > +DOCKER_TOOLS := travis
> > +
> > +TESTS ?= %
> > +IMAGES ?= %
> > +
> > +$(foreach i,$(DOCKER_IMAGES), \
> > + $(eval docker-image: docker-image-$i) \
> > + $(eval docker-image-$i: IMAGE = $i) \
> > + $(eval .PHONY: docker-image-$i address@hidden) \
> > + $(foreach t,$(DOCKER_TESTS) $(DOCKER_TOOLS), \
> > + $(eval .PHONY: address@hidden) \
> > + $(eval address@hidden: docker-image-$i) \
> > + $(eval address@hidden: address@hidden) \
> > + $(eval address@hidden: address@hidden) \
> > + $(eval docker-test: address@hidden) \
> > + ) \
> > +)
> > +
> > +docker:
> > + @echo 'Building QEMU and running 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 ' address@hidden: Run all tests in container "IMAGE".'
> > + @echo ' docker-image: Build all images.'
> > + @echo ' docker-image-IMAGE: Build image "IMG".'
> > + @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 ' 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)'
>
> OK none of these test targets seem to be working for me:
>
> COPY SRC /tmp/qemu-src.JCUng
> RUN test-clang in centos6
> Prerequisite 'clang' not present, skip
> RUN test-mingw in centos6
> Prerequisite 'mingw' not present, skip
> RUN test-quick in centos6
> make: *** address@hidden Error 2
>
> Running with V=1 doesn't elucidate me much further:
>
> /home/alex/lsrc/qemu/qemu.git/rules.mak:178: warning: overriding commands for
> target `clean-timestamp'
> /home/alex/lsrc/qemu/qemu.git/rules.mak:178: warning: ignoring old commands
> for target `clean-timestamp'
> mkdir -p /tmp/qemu-src.HtY32 && (if git diff-index --quiet HEAD --
> &>/dev/null; then git archive -1 HEAD --format=tar.gz; else git archive -1
> $(git stash create) --format=tar.gz; fi) | tar -xz -C /tmp/qemu-src.HtY32
> /home/alex/lsrc/qemu/qemu.git/tests/docker/docker_run --rm --privileged -t
> --net=none -v $(realpath /home/alex/lsrc/qemu/qemu.git):/var/tmp/qemu -e
> QEMU_SRC=/var/tmp/qemu -e V=1 -e J= -e PAUSE= -v
> /var/tmp/qemu-docker-ccache:/var/tmp/ccache -e CCACHE_DIR=/var/tmp/ccache
> qemu:centos6 /var/tmp/qemu/tests/docker/run
> /var/tmp/qemu/tests/docker/test-clang;
> Prerequisite 'clang' not present, skip
> /home/alex/lsrc/qemu/qemu.git/tests/docker/docker_run --rm --privileged -t
> --net=none -v $(realpath /home/alex/lsrc/qemu/qemu.git):/var/tmp/qemu -e
> QEMU_SRC=/var/tmp/qemu -e V=1 -e J= -e PAUSE= -v
> /var/tmp/qemu-docker-ccache:/var/tmp/ccache -e CCACHE_DIR=/var/tmp/ccache
> qemu:centos6 /var/tmp/qemu/tests/docker/run
> /var/tmp/qemu/tests/docker/test-mingw;
> Prerequisite 'mingw' not present, skip
> /home/alex/lsrc/qemu/qemu.git/tests/docker/docker_run --rm --privileged -t
> --net=none -v $(realpath /home/alex/lsrc/qemu/qemu.git):/var/tmp/qemu -e
> QEMU_SRC=/var/tmp/qemu -e V=1 -e J= -e PAUSE= -v
> /var/tmp/qemu-docker-ccache:/var/tmp/ccache -e CCACHE_DIR=/var/tmp/ccache
> qemu:centos6 /var/tmp/qemu/tests/docker/run
> /var/tmp/qemu/tests/docker/test-quick;
> make: *** address@hidden Error 2
>
> If I drop to the shell using:
>
> make address@hidden V=1 PAUSE=1
>
> And run:
>
> /var/tmp/qemu/tests/docker/test-quick
>
> I get a cryptic silent fail. I think we need more verbosity (select-able
> when required) to see whats going on in the depths.
>
> > +
> > +docker-image-%:
> > + @if test -z "$(IMAGE)"; then echo "Invalid target"; exit 1; fi
> > + $(if $(filter $(IMAGES),$(IMAGE)), $(call quiet-command,\
> > + $(SRC_PATH)/tests/docker/docker_build qemu:$(IMAGE) \
> > + $(SRC_PATH)/tests/docker/dockerfiles/$(IMAGE).docker \
> > + $(if $V,-v,), " BUILD $(IMAGE)"))
> > +
> > +
> > +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-%:
> > + @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_run $(if $(KEEP),,--rm)
> > \
> > + --privileged -t --net=none \
> > + -v $$(realpath $(SRC_PATH)):/var/tmp/qemu \
> > + -e QEMU_SRC=/var/tmp/qemu \
> > + -e V=$V -e J=$J \
> > + -v /var/tmp/qemu-docker-ccache:/var/tmp/ccache \
> > + -e CCACHE_DIR=/var/tmp/ccache \
> > + qemu:$(IMAGE) \
> > + /var/tmp/qemu/tests/docker/run \
> > + /var/tmp/qemu/tests/docker/$(CMD); \
> > + , " RUN $(CMD) in $(IMAGE)")))
> > +
> > +docker-clean:
> > + $(call quiet-command, $(SRC_PATH)/tests/docker/docker_clean)
>
>
> --
> Alex Bennée
- [Qemu-devel] [PATCH v2 00/15] tests: Introducing docker tests, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 01/15] tests: Add utilities for docker testing, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 02/15] Makefile: Rules for docker testing, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 03/15] docker: Add images, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 04/15] docker: Add test runner, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 06/15] docker: Add quick test, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 05/15] docker: Add common.rc, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 07/15] docker: Add full test, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 08/15] docker: Add clang test, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 09/15] docker: Add mingw test, Fam Zheng, 2016/02/16
- [Qemu-devel] [PATCH v2 11/15] docs: Add text for tests/docker in build-system.txt, Fam Zheng, 2016/02/16