[Qemu-devel] [PATCH 0/2] qemu-iotests: add a Travis job

Cleber Rosa posted 2 patches 5 years, 2 months ago
Test docker-mingw@fedora passed
Test asan passed
Test checkpatch passed
Test docker-clang@ubuntu passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190209040132.25573-1-crosa@redhat.com
Maintainers: Max Reitz <mreitz@redhat.com>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, Kevin Wolf <kwolf@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Fam Zheng <fam@euphon.net>
.travis.yml            | 8 ++++++++
tests/qemu-iotests/235 | 5 +++--
tests/qemu-iotests/238 | 5 +++--
3 files changed, 14 insertions(+), 4 deletions(-)
[Qemu-devel] [PATCH 0/2] qemu-iotests: add a Travis job
Posted by Cleber Rosa 5 years, 2 months ago
Besides the obvious reasons of testing more, and somewhat for free,
running the qemu-iotests along the other tests on Travis also makes
sure that changes to shared code such as "scripts/qemu.py" and the
like won't break other users of the same code.

Cleber Rosa (2):
  qemu-iotests: only enable kvm if the host seems to support it
  qemu-iotests: add "make check-block" job to Travis

 .travis.yml            | 8 ++++++++
 tests/qemu-iotests/235 | 5 +++--
 tests/qemu-iotests/238 | 5 +++--
 3 files changed, 14 insertions(+), 4 deletions(-)

-- 
2.20.1


Re: [Qemu-devel] [PATCH 0/2] qemu-iotests: add a Travis job
Posted by Cleber Rosa 5 years, 2 months ago

On 2/8/19 11:01 PM, Cleber Rosa wrote:
> Besides the obvious reasons of testing more, and somewhat for free,
> running the qemu-iotests along the other tests on Travis also makes
> sure that changes to shared code such as "scripts/qemu.py" and the
> like won't break other users of the same code.
> 
> Cleber Rosa (2):
>   qemu-iotests: only enable kvm if the host seems to support it
>   qemu-iotests: add "make check-block" job to Travis
> 
>  .travis.yml            | 8 ++++++++
>  tests/qemu-iotests/235 | 5 +++--
>  tests/qemu-iotests/238 | 5 +++--
>  3 files changed, 14 insertions(+), 4 deletions(-)
> 

Just a follow up showing the results for the added Travis job:

https://travis-ci.org/clebergnu/qemu/jobs/490806941#L2321

Cheers,
- Cleber.