mbox series

[kvm-unit-test,0/5] Improvements for the Travis CI

Message ID 20191113112649.14322-1-thuth@redhat.com (mailing list archive)
Headers show
Series Improvements for the Travis CI | expand

Message

Thomas Huth Nov. 13, 2019, 11:26 a.m. UTC
The first two patches make the test matrix a little bit more flexible,
and the fourth patch enables the 32-bit builds on x86.

But the most important patch is likely the third one: It is possible to
test with KVM on Travis now, so we can run the tests within a real KVM
environment, without TCG! The only caveat is that qemu-system-x86_64
has to run as root ... fixing only the permissions of /dev/kvm did
not help here, I still got a "Permission denied" in that case.

Thomas Huth (5):
  travis.yml: Re-arrange the test matrix
  travis.yml: Install only the required packages for each entry in the
    matrix
  travis.yml: Test with KVM instead of TCG (on x86)
  travis.yml: Test the i386 build, too
  travis.yml: Expect that at least one test succeeds

 .travis.yml | 155 +++++++++++++++++++++++++++++++++++-----------------
 1 file changed, 104 insertions(+), 51 deletions(-)

Comments

Paolo Bonzini Nov. 13, 2019, 2:50 p.m. UTC | #1
On 13/11/19 12:26, Thomas Huth wrote:
> The first two patches make the test matrix a little bit more flexible,
> and the fourth patch enables the 32-bit builds on x86.
> 
> But the most important patch is likely the third one: It is possible to
> test with KVM on Travis now, so we can run the tests within a real KVM
> environment, without TCG! The only caveat is that qemu-system-x86_64
> has to run as root ... fixing only the permissions of /dev/kvm did
> not help here, I still got a "Permission denied" in that case.

Ah, that's Debian.  You need to be in group kvm or root to run KVM.

Looks good, can you include it in the next pull request?

Paolo

> Thomas Huth (5):
>   travis.yml: Re-arrange the test matrix
>   travis.yml: Install only the required packages for each entry in the
>     matrix
>   travis.yml: Test with KVM instead of TCG (on x86)
>   travis.yml: Test the i386 build, too
>   travis.yml: Expect that at least one test succeeds
> 
>  .travis.yml | 155 +++++++++++++++++++++++++++++++++++-----------------
>  1 file changed, 104 insertions(+), 51 deletions(-)
>
David Hildenbrand Nov. 14, 2019, 12:29 p.m. UTC | #2
On 13.11.19 12:26, Thomas Huth wrote:
> The first two patches make the test matrix a little bit more flexible,
> and the fourth patch enables the 32-bit builds on x86.
> 
> But the most important patch is likely the third one: It is possible to
> test with KVM on Travis now, so we can run the tests within a real KVM
> environment, without TCG! The only caveat is that qemu-system-x86_64
> has to run as root ... fixing only the permissions of /dev/kvm did
> not help here, I still got a "Permission denied" in that case.
> 
> Thomas Huth (5):
>    travis.yml: Re-arrange the test matrix
>    travis.yml: Install only the required packages for each entry in the
>      matrix
>    travis.yml: Test with KVM instead of TCG (on x86)
>    travis.yml: Test the i386 build, too
>    travis.yml: Expect that at least one test succeeds
> 
>   .travis.yml | 155 +++++++++++++++++++++++++++++++++++-----------------
>   1 file changed, 104 insertions(+), 51 deletions(-)
> 

As discussed, queued to

https://github.com/davidhildenbrand/kvm-unit-tests.git s390x-next

for now. PULL request early next week.