[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/7] tests/docker: add basic user mapping sup
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/7] tests/docker: add basic user mapping support |
Date: |
Thu, 26 Jan 2017 17:08:29 +0800 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Tue, 01/24 14:33, Alex Bennée wrote:
> Currently all docker builds are done by exporting a tarball to the
> docker container and running the build as the containers root user.
> Other use cases are possible however and it is possible to map a part
> of users file-system to the container. This is useful for example for
> doing cross-builds of arbitrary source trees. For this to work
> smoothly the container needs to have a user created that maps cleanly
> to the host system.
>
> This adds a -u option to the docker script so that:
>
> DEB_ARCH=armhf DEB_TYPE=stable ./tests/docker/docker.py build \
> -u --include-executable=arm-linux-user/qemu-arm \
> debian:armhf ./tests/docker/dockerfiles/debian-bootstrap.docker
>
> Will build a container that can then be run like:
>
> docker run --rm -it -v /home/alex/lsrc/qemu/risu.git/:/src \
> --user=alex:alex -w /src/ debian:armhf \
> sh -c "make clean && ./configure -s && make"
>
> Signed-off-by: Alex Bennée <address@hidden>
>
> ---
> v2
> - write the useradd directly
> - change long option to --add-current-user
> ---
> tests/docker/docker.py | 15 +++++++++++++--
> 1 file changed, 13 insertions(+), 2 deletions(-)
>
> diff --git a/tests/docker/docker.py b/tests/docker/docker.py
> index 37d83199e7..b85739a9e2 100755
> --- a/tests/docker/docker.py
> +++ b/tests/docker/docker.py
> @@ -25,6 +25,7 @@ import signal
> from tarfile import TarFile, TarInfo
> from StringIO import StringIO
> from shutil import copy, rmtree
> +from pwd import getpwuid
>
>
> DEVNULL = open(os.devnull, 'wb')
> @@ -149,13 +150,20 @@ class Docker(object):
> labels = json.loads(resp)[0]["Config"].get("Labels", {})
> return labels.get("com.qemu.dockerfile-checksum", "")
>
> - def build_image(self, tag, docker_dir, dockerfile, quiet=True,
> argv=None):
> + def build_image(self, tag, docker_dir, dockerfile,
> + quiet=True, user=False, argv=None):
> if argv == None:
> argv = []
>
> tmp_df = tempfile.NamedTemporaryFile(dir=docker_dir,
> suffix=".docker")
> tmp_df.write(dockerfile)
>
> + if user:
> + uid = os.getuid()
> + uname = getpwuid(uid).pw_name
> + tmp_df.write("\n")
> + tmp_df.write("RUN useradd -u %d -U %s" % (uid, uname))
> +
> tmp_df.write("\n")
> tmp_df.write("LABEL com.qemu.dockerfile-checksum=%s" %
> _text_checksum(dockerfile))
> @@ -225,6 +233,9 @@ class BuildCommand(SubCommand):
> help="""Specify a binary that will be copied to
> the
> container together with all its dependent
> libraries""")
> + parser.add_argument("--add-current-user", "-u", dest="user",
> + action="store_true",
> + help="Add the current user to images passwd")
s/images/image's/ ?
otherwise looks good. If you fix that, you can add
Reviewed-by: Fam Zheng <address@hidden>
> parser.add_argument("tag",
> help="Image Tag")
> parser.add_argument("dockerfile",
> @@ -261,7 +272,7 @@ class BuildCommand(SubCommand):
> docker_dir)
>
> dkr.build_image(tag, docker_dir, dockerfile,
> - quiet=args.quiet, argv=argv)
> + quiet=args.quiet, user=args.user, argv=argv)
>
> rmtree(docker_dir)
>
> --
> 2.11.0
>
>