[v4] docker: Add test-block

[Qemu-devel] [PATCH v4 0/6] docker: Add test-block

Fam Zheng posted 6 patches 6 years, 7 months ago

Diff against v1 v2 v3
Download series mbox

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

Test checkpatch passed

Test docker passed

Test s390x passed

tests/docker/Makefile.include          |  1 +
tests/docker/common.rc                 | 15 +++++++++++++--
tests/docker/dockerfiles/fedora.docker |  1 +
tests/docker/run                       |  8 +++-----
tests/docker/test-block                | 22 ++++++++++++++++++++++
5 files changed, 40 insertions(+), 7 deletions(-)
create mode 100755 tests/docker/test-block

Expand all Fold all

[Qemu-devel] [PATCH v4 0/6] docker: Add test-block

Posted by Fam Zheng 6 years, 7 months ago

Based-on: 20170905021201.25684-1-famz@redhat.com
([PATCH v6 00/12] tests: Add VM based build tests (for non-x86_64 and/or
non-Linux))

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.

v4: Don't use 'set -e'. [Eric]
    Add Kashyap's rev-by to unchanged patches.

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 (6):
  docker: Fix return code of build_qemu()
  docker: Add test_fail and prep_fail
  docker: Use unconfined security profile
  docker: Add nettle-devel to fedora image
  docker: Add test-block
  docker: Drop 'set -e' from run script

 tests/docker/Makefile.include          |  1 +
 tests/docker/common.rc                 | 15 +++++++++++++--
 tests/docker/dockerfiles/fedora.docker |  1 +
 tests/docker/run                       |  8 +++-----
 tests/docker/test-block                | 22 ++++++++++++++++++++++
 5 files changed, 40 insertions(+), 7 deletions(-)
 create mode 100755 tests/docker/test-block

-- 
2.13.5

Re: [Qemu-devel] [PATCH v4 0/6] docker: Add test-block

Posted by Stefan Hajnoczi 6 years, 7 months ago

On Tue, Sep 05, 2017 at 10:56:08AM +0800, Fam Zheng wrote:
> Based-on: 20170905021201.25684-1-famz@redhat.com
> ([PATCH v6 00/12] tests: Add VM based build tests (for non-x86_64 and/or
> non-Linux))
> 
> 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.
> 
> v4: Don't use 'set -e'. [Eric]
>     Add Kashyap's rev-by to unchanged patches.
> 
> 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 (6):
>   docker: Fix return code of build_qemu()
>   docker: Add test_fail and prep_fail
>   docker: Use unconfined security profile
>   docker: Add nettle-devel to fedora image
>   docker: Add test-block
>   docker: Drop 'set -e' from run script
> 
>  tests/docker/Makefile.include          |  1 +
>  tests/docker/common.rc                 | 15 +++++++++++++--
>  tests/docker/dockerfiles/fedora.docker |  1 +
>  tests/docker/run                       |  8 +++-----
>  tests/docker/test-block                | 22 ++++++++++++++++++++++
>  5 files changed, 40 insertions(+), 7 deletions(-)
>  create mode 100755 tests/docker/test-block
> 
> -- 
> 2.13.5
> 
> 

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>