[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/6] docker.py: add --run-as-current-user
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/6] docker.py: add --run-as-current-user |
Date: |
Mon, 15 Jul 2019 18:43:28 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 7/13/19 4:33 PM, Marc-André Lureau wrote:
> Podman will need further tweaks.
>
> Suggested-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> tests/docker/Makefile.include | 2 +-
> tests/docker/docker.py | 5 +++++
> 2 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index aaf5396b85..f4226b79d0 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -212,7 +212,7 @@ docker-run: docker-qemu-src
> " COPYING $(EXECUTABLE) to $(IMAGE)"))
> $(call quiet-command, \
> $(DOCKER_SCRIPT) run \
> - $(if $(NOUSER),,-u $(shell id -u)) \
> + $(if $(NOUSER),,--run-as-current-user) \
> --security-opt seccomp=unconfined \
> $(if $V,,--rm) \
> $(if $(DEBUG),-ti,) \
> diff --git a/tests/docker/docker.py b/tests/docker/docker.py
> index 53a8c9c801..f15545aeea 100755
> --- a/tests/docker/docker.py
> +++ b/tests/docker/docker.py
> @@ -333,8 +333,13 @@ class RunCommand(SubCommand):
> def args(self, parser):
> parser.add_argument("--keep", action="store_true",
> help="Don't remove image when command completes")
> + parser.add_argument("--run-as-current-user", action="store_true",
> + help="Run container using the current user's
> uid")
>
> def run(self, args, argv):
> + if args.run_as_current_user:
> + uid = os.getuid()
> + argv = [ "-u", str(uid) ] + argv
> return Docker().run(argv, args.keep, quiet=args.quiet)
>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
- [Qemu-devel] [PATCH v3 0/6] tests/docker: add podman support, Marc-André Lureau, 2019/07/13
- [Qemu-devel] [PATCH v3 1/6] docker.py: add --run-as-current-user, Marc-André Lureau, 2019/07/13
- [Qemu-devel] [PATCH v3 2/6] docker.py: add podman support, Marc-André Lureau, 2019/07/13
- [Qemu-devel] [PATCH v3 3/6] tests/docker: add podman support, Marc-André Lureau, 2019/07/13
- [Qemu-devel] [PATCH v3 4/6] tests: specify the address family when checking bind, Marc-André Lureau, 2019/07/13
- [Qemu-devel] [PATCH v3 5/6] test-char: skip tcp tests if ipv4 check failed, Marc-André Lureau, 2019/07/13