[Qemu-devel] [PATCH v3 0/4] Add a CentOS test image to run docker tests

Fam Zheng posted 4 patches 5 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180712012829.20231-1-famz@redhat.com
Test checkpatch passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
tests/docker/Makefile.include |  7 ++-
tests/vm/Makefile.include     |  3 +-
tests/vm/basevm.py            | 10 ++++-
tests/vm/centos               | 84 +++++++++++++++++++++++++++++++++++
tests/vm/freebsd              |  4 +-
tests/vm/netbsd               |  4 +-
tests/vm/openbsd              |  4 +-
tests/vm/ubuntu.i386          |  4 +-
8 files changed, 107 insertions(+), 13 deletions(-)
create mode 100755 tests/vm/centos
[Qemu-devel] [PATCH v3 0/4] Add a CentOS test image to run docker tests
Posted by Fam Zheng 5 years, 9 months ago
v3: Add 'make vm-test' document. [Phil]

v2: Drop archive-source.sh changes.
    The new test depends on the iotests nbd fix I posted today to pass.

Docker testing on patchew has long suffered from 'make check' hangings. The
cleanness of VM testing is the cure. Now let's add a CentOS 7 image to run the
tests.  It's purely ad-hoc, but hopefully still easy to understand and use for
everyone.

The first patch makes passing source code from host to the container in VM
working, and is a nice clean up.

The second patch makes caches work, to speed up repetitive runs like on
patchew.

The last patch adds the new image that does the job. Two out of three docker
tests running on patchew.org are added to the image. I'll wait for Peter to fix
the 'docker-test-quick@centos6' hanging (oob test) before adding it too.

Fam

Fam Zheng (4):
  tests: Add an option for snapshot (default: off)
  tests/vm: Pass verbose flag into VM make commands
  tests: Allow overriding archive path with SRC_ARCHIVE
  tests: Add centos VM testing

 tests/docker/Makefile.include |  7 ++-
 tests/vm/Makefile.include     |  3 +-
 tests/vm/basevm.py            | 10 ++++-
 tests/vm/centos               | 84 +++++++++++++++++++++++++++++++++++
 tests/vm/freebsd              |  4 +-
 tests/vm/netbsd               |  4 +-
 tests/vm/openbsd              |  4 +-
 tests/vm/ubuntu.i386          |  4 +-
 8 files changed, 107 insertions(+), 13 deletions(-)
 create mode 100755 tests/vm/centos

-- 
2.17.1


Re: [Qemu-devel] [PATCH v3 0/4] Add a CentOS test image to run docker tests
Posted by Philippe Mathieu-Daudé 5 years, 9 months ago
On 07/11/2018 10:28 PM, Fam Zheng wrote:
> v3: Add 'make vm-test' document. [Phil]
> 
> v2: Drop archive-source.sh changes.
>     The new test depends on the iotests nbd fix I posted today to pass.
> 
> Docker testing on patchew has long suffered from 'make check' hangings. The
> cleanness of VM testing is the cure. Now let's add a CentOS 7 image to run the
> tests.  It's purely ad-hoc, but hopefully still easy to understand and use for
> everyone.
> 
> The first patch makes passing source code from host to the container in VM
> working, and is a nice clean up.
> 
> The second patch makes caches work, to speed up repetitive runs like on
> patchew.
> 
> The last patch adds the new image that does the job. Two out of three docker
> tests running on patchew.org are added to the image. I'll wait for Peter to fix
> the 'docker-test-quick@centos6' hanging (oob test) before adding it too.
> 
> Fam
> 
> Fam Zheng (4):
>   tests: Add an option for snapshot (default: off)
>   tests/vm: Pass verbose flag into VM make commands
>   tests: Allow overriding archive path with SRC_ARCHIVE
>   tests: Add centos VM testing

Series:
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> 
>  tests/docker/Makefile.include |  7 ++-
>  tests/vm/Makefile.include     |  3 +-
>  tests/vm/basevm.py            | 10 ++++-
>  tests/vm/centos               | 84 +++++++++++++++++++++++++++++++++++
>  tests/vm/freebsd              |  4 +-
>  tests/vm/netbsd               |  4 +-
>  tests/vm/openbsd              |  4 +-
>  tests/vm/ubuntu.i386          |  4 +-
>  8 files changed, 107 insertions(+), 13 deletions(-)
>  create mode 100755 tests/vm/centos
>