Re: [Qemu-devel] [PATCH v3 3/6] tests/docker: add podman support

[Alex Bennée]

Marc-André Lureau <address@hidden> writes:

> Allow to specify the container engine to run with ENGINE variable.
>
> By default, ENGINE=auto and will select either podman or docker.
>
> Signed-off-by: Marc-André Lureau <address@hidden>

Reviewed-by: Alex Bennée <address@hidden>

> ---
>  Makefile                      | 2 +-
>  tests/docker/Makefile.include | 8 ++++++--
>  2 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 1fcbaed62c..7863bb0cf5 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1153,7 +1153,7 @@ endif
>       @echo  ''
>       @echo  'Test targets:'
>       @echo  '  check           - Run all tests (check-help for details)'
> -     @echo  '  docker          - Help about targets running tests inside 
> Docker containers'
> +     @echo  '  docker          - Help about targets running tests inside 
> containers'
>       @echo  '  vm-help         - Help about targets running tests inside VM'
>       @echo  ''
>       @echo  'Documentation targets:'
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index f4226b79d0..3c1aec862a 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -17,7 +17,9 @@ DOCKER_TESTS := $(notdir $(shell \
>
>  DOCKER_TOOLS := travis
>
> -DOCKER_SCRIPT=$(SRC_PATH)/tests/docker/docker.py
> +ENGINE := auto
> +
> +DOCKER_SCRIPT=$(SRC_PATH)/tests/docker/docker.py --engine $(ENGINE)
>
>  TESTS ?= %
>  IMAGES ?= %
> @@ -146,7 +148,7 @@ $(foreach i,$(filter-out 
> $(DOCKER_PARTIAL_IMAGES),$(DOCKER_IMAGES) $(DOCKER_DEPR
>  )
>
>  docker:
> -     @echo 'Build QEMU and run tests inside Docker containers'
> +     @echo 'Build QEMU and run tests inside Docker or Podman containers'
>       @echo
>       @echo 'Available targets:'
>       @echo
> @@ -193,6 +195,8 @@ endif
>       @echo '    EXECUTABLE=<path>    Include executable in image.'
>       @echo '    EXTRA_FILES="<path> [... <path>]"'
>       @echo '                         Include extra files in image.'
> +     @echo '    ENGINE=auto/docker/podman'
> +     @echo '                         Specify which container engine to run.'
>
>  # This rule if for directly running against an arbitrary docker target.
>  # It is called by the expanded docker targets (e.g. make


--
Alex Bennée