[v3] docker: Add test-block

[Qemu-devel] [PATCH v3 0/3] docker: Add test-block

Fam Zheng posted 3 patches 8 years, 2 months ago

Diff against v1 v2 v4
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170901152200.5953-1-famz@redhat.com

Test FreeBSD passed

Test checkpatch passed

Test docker passed

Test s390x passed

There is a newer version of this series

tests/docker/Makefile.include          |  1 +
tests/docker/dockerfiles/fedora.docker |  1 +
tests/docker/test-block                | 22 ++++++++++++++++++++++
3 files changed, 24 insertions(+)
create mode 100755 tests/docker/test-block

Expand all Fold all

[Qemu-devel] [PATCH v3 0/3] docker: Add test-block

Posted by Fam Zheng 8 years, 2 months ago

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.

v3: Add nbd, luks and limit to quick group.

v2: Add nettle-devel because a few iotest cases blindly assumes the support is
there.

Fam Zheng (3):
  docker: Use unconfined security profile
  docker: Add nettle-devel to fedora image
  docker: Add test-block

 tests/docker/Makefile.include          |  1 +
 tests/docker/dockerfiles/fedora.docker |  1 +
 tests/docker/test-block                | 22 ++++++++++++++++++++++
 3 files changed, 24 insertions(+)
 create mode 100755 tests/docker/test-block

-- 
2.13.5

Re: [Qemu-devel] [PATCH v3 0/3] docker: Add test-block

Posted by Kashyap Chamarthy 8 years, 2 months ago

On Fri, Sep 01, 2017 at 11:21:57PM +0800, 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.
> 
> v3: Add nbd, luks and limit to quick group.
> 
> v2: Add nettle-devel because a few iotest cases blindly assumes the support is
> there.
> 
> Fam Zheng (3):
>   docker: Use unconfined security profile
>   docker: Add nettle-devel to fedora image
>   docker: Add test-block

All three of the patches look good to me.  FWIW:

Reviewed-by: Kashyap Chamarthy <kchamart@redhat.com>


>  tests/docker/Makefile.include          |  1 +
>  tests/docker/dockerfiles/fedora.docker |  1 +
>  tests/docker/test-block                | 22 ++++++++++++++++++++++
>  3 files changed, 24 insertions(+)
>  create mode 100755 tests/docker/test-block
> 
> -- 
> 2.13.5
> 

-- 
/kashyap

Re: [Qemu-devel] [PATCH v3 0/3] docker: Add test-block

Posted by Eric Blake 8 years, 2 months ago

On 09/01/2017 10:21 AM, 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.
> 
> v3: Add nbd, luks and limit to quick group.
> 
> v2: Add nettle-devel because a few iotest cases blindly assumes the support is
> there.
> 
> Fam Zheng (3):
>   docker: Use unconfined security profile
>   docker: Add nettle-devel to fedora image
>   docker: Add test-block

Based-on: <20170901105434.3288-1-berrange@redhat.com>
([PATCH 0/2] Fix / skip recent iotests with LUKS driver)

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org