[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/12] tests: Introducing docker tests
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH 00/12] tests: Introducing docker tests |
Date: |
Mon, 15 Feb 2016 17:59:04 +0000 |
User-agent: |
mu4e 0.9.17; emacs 25.0.91.3 |
Fam Zheng <address@hidden> writes:
> v1: Since RFC, addressed comments from reviewers, and improved a lot of
> things.
> Thanks to Daniel, Eric, Paolo, Stefan, for the feedback.
>
> This series adds a new "docker" make target family to run tests in created
> docker containers.
>
> To begin with, this can be a place to store standard env/command combinations
> to
> build and test QEMU.
I think I'm done for this review pass. Please CC me on your next
version.
I've run a quick test on my ARM64 Debian Jessie box which predictably
failed. It looks like this is a docker hub issue for one thing as:
docker pull debian:stable
creates an x86_64 image on the arm64 box which predictably fails. I
suspect the only solution is going to be storing our own built images on
the docker hub and tweaking the tooling to take host architecture into
account.
Do you have any non-x86 boxes to test against or are you going to need
some help with non-x86 support?
Cheers,
--
Alex Bennée
- [Qemu-devel] [PATCH 08/12] docker: Add mingw test, (continued)
- [Qemu-devel] [PATCH 08/12] docker: Add mingw test, Fam Zheng, 2016/02/05
- [Qemu-devel] [PATCH 09/12] docker: Add travis tool, Fam Zheng, 2016/02/05
- [Qemu-devel] [PATCH 10/12] docs: Add text for tests/docker in build-system.txt, Fam Zheng, 2016/02/05
- [Qemu-devel] [PATCH 11/12] .gitignore: Ignore temporary dockerfile, Fam Zheng, 2016/02/05
- [Qemu-devel] [PATCH 12/12] MAINTAINERS: Add tests/docker, Fam Zheng, 2016/02/05
- Re: [Qemu-devel] [PATCH 00/12] tests: Introducing docker tests, Alex Bennée, 2016/02/10
- Re: [Qemu-devel] [PATCH 00/12] tests: Introducing docker tests,
Alex Bennée <=