[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 3/8] tests/docker/docker.py: check and run .p
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v6 3/8] tests/docker/docker.py: check and run .pre script |
Date: |
Tue, 19 Jul 2016 10:29:39 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.0.95.10 |
Fam Zheng <address@hidden> writes:
> From: Alex Bennée <address@hidden>
>
> The docker script will now search for an associated $dockerfile.pre
> script which gets run in the same build context as the dockerfile will
> be. This is to support pre-seeding the build context before running the
> docker build.
>
> Signed-off-by: Alex Bennée <address@hidden>
> Message-id: address@hidden
> ---
> tests/docker/docker.py | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/tests/docker/docker.py b/tests/docker/docker.py
> index 96d906e..b61d934 100755
> --- a/tests/docker/docker.py
> +++ b/tests/docker/docker.py
> @@ -221,6 +221,18 @@ class BuildCommand(SubCommand):
> # Create a docker context directory for the build
> docker_dir = tempfile.mkdtemp(prefix="docker_build")
>
> + # Is there a .pre file to run in the build context?
> + docker_pre = os.path.splitext(args.dockerfile)[0]+".pre"
> + if os.path.exists(docker_pre):
> + rc = subprocess.call(os.path.realpath(docker_pre),
> + cwd=docker_dir)
> + if rc == 3:
> + print "Skip"
> + return 0
> + else:
> + print "%s exited with code %d" % (docker_pre, rc)
> + return 1
> +
We need to let an exit of 0 through ;-)
> # Do we include a extra binary?
> if args.include_executable:
> _copy_binary_with_libs(args.include_executable,
--
Alex Bennée
- [Qemu-devel] [PATCH v6 0/8] docker: Support building qemu-user powered docker test images, Fam Zheng, 2016/07/19
- [Qemu-devel] [PATCH v6 3/8] tests/docker/docker.py: check and run .pre script, Fam Zheng, 2016/07/19
- Re: [Qemu-devel] [PATCH v6 3/8] tests/docker/docker.py: check and run .pre script,
Alex Bennée <=
- [Qemu-devel] [PATCH v6 2/8] tests/docker/docker.py: support --include-executable, Fam Zheng, 2016/07/19
- [Qemu-devel] [PATCH v6 6/8] docker: More sensible run script, Fam Zheng, 2016/07/19
- [Qemu-devel] [PATCH v6 1/8] tests/docker/docker.py: docker_dir outside build, Fam Zheng, 2016/07/19
- [Qemu-devel] [PATCH v6 4/8] tests/docker/dockerfiles: new debian-bootstrap.docker, Fam Zheng, 2016/07/19
- [Qemu-devel] [PATCH v6 5/8] tests/docker/docker.py: add update operation, Fam Zheng, 2016/07/19
- [Qemu-devel] [PATCH v6 7/8] docker: Fix exit code if $CMD failed, Fam Zheng, 2016/07/19