[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 0/8] docker: Support building qemu-user power
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v6 0/8] docker: Support building qemu-user powered docker test images |
Date: |
Tue, 19 Jul 2016 11:59:18 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.0.95.10 |
Fam Zheng <address@hidden> writes:
> v6: A few tweaks to make this more mergable. Including:
> - Add and catch magic pre script exit code "3" to allow skipping image.
> - Tweak debian-bootstrap.pre to return '3' when appropriate (It should
> still return 1 for debootstrap command failure).
> - Fix "realpath" to "readlink -e" to be compatible with centos6.
> - Add the last patch to skip test if the image is skipped by .pre.
>
> Alex, if this looks good to you, I can send a pull req today.
I've rolled the fixes into v7:
https://github.com/stsquad/qemu/tree/review/docker-user-v7
>
> Alex Bennée (5):
> tests/docker/docker.py: docker_dir outside build
> tests/docker/docker.py: support --include-executable
> tests/docker/docker.py: check and run .pre script
> tests/docker/dockerfiles: new debian-bootstrap.docker
> tests/docker/docker.py: add update operation
>
> Fam Zheng (3):
> docker: More sensible run script
> docker: Fix exit code if $CMD failed
> docker: Don't start a container that doesn't exist
>
> tests/docker/Makefile.include | 9 +-
> tests/docker/docker.py | 150
> +++++++++++++++++++++--
> tests/docker/dockerfiles/debian-bootstrap.docker | 21 ++++
> tests/docker/dockerfiles/debian-bootstrap.pre | 87 +++++++++++++
> tests/docker/run | 16 ++-
> 5 files changed, 267 insertions(+), 16 deletions(-)
> create mode 100644 tests/docker/dockerfiles/debian-bootstrap.docker
> create mode 100755 tests/docker/dockerfiles/debian-bootstrap.pre
--
Alex Bennée
- Re: [Qemu-devel] [PATCH v6 2/8] tests/docker/docker.py: support --include-executable, (continued)
- [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
- [Qemu-devel] [PATCH v6 8/8] docker: Don't start a container that doesn't exist, Fam Zheng, 2016/07/19
- Re: [Qemu-devel] [PATCH v6 0/8] docker: Support building qemu-user powered docker test images, Alex Bennée, 2016/07/19
- Re: [Qemu-devel] [PATCH v6 0/8] docker: Support building qemu-user powered docker test images,
Alex Bennée <=