[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] docker: Add test-block
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] docker: Add test-block |
Date: |
Fri, 5 May 2017 16:05:13 +0800 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Fri, 05/05 09:44, Paolo Bonzini wrote:
>
>
> On 05/05/2017 05:23, Fam Zheng wrote:
> > A few tweaks to the docker images and running commands allow us to run
> > qemu-iotests in the fedora container. This will be added to the patchew test
> > cases later.
> >
> > Fam Zheng (5):
> > docker: Run tests with current user
> > docker: Add bzip2 and hostname to fedora image
> > docker: Use unconfined security profile
> > docker: Add libaio to fedora image
> > docker: Add test-block
> >
> > tests/docker/Makefile.include | 3 ++-
> > tests/docker/dockerfiles/fedora.docker | 4 ++--
> > tests/docker/test-block | 21 +++++++++++++++++++++
> > 3 files changed, 25 insertions(+), 3 deletions(-)
> > create mode 100755 tests/docker/test-block
> >
>
> Patches 1-4:
>
> Reviewed-by: Paolo Bonzini <address@hidden>
>
> For patch 5, I think we should revive the "quick" qemu-iotests group
> (maybe we can have support for skipping groups, and have a "slow" group
> with the very few tests that take >30 seconds?) and add the quick tests
> to "make check".
Sounds like a good idea, but for a separate series, IMO.
The idea of the new test-block is to include all qemu-iotests tests. I only
added raw and qcow2 because vmdk etc. fails case 181. I could use "-x" but
that's a bit ugly - they can wait for the fix.
Fam
- Re: [Qemu-devel] [PATCH 3/5] docker: Use unconfined security profile, (continued)