[Qemu-devel] [PATCH 0/2] docker: allow running "docker-travis" with arbitrary parallelism

Paolo Bonzini posted 2 patches 6 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170714093407.11526-1-pbonzini@redhat.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
.travis.yml            | 3 ++-
tests/docker/travis.py | 6 +++---
2 files changed, 5 insertions(+), 4 deletions(-)
[Qemu-devel] [PATCH 0/2] docker: allow running "docker-travis" with arbitrary parallelism
Posted by Paolo Bonzini 6 years, 9 months ago
docker-travis is limited to the -j3 flag specified for running on the
SaaS Travis CI infrastructure.

When running the test matrix locally with docker, however, we can run
much more jobs at the same time on a suitably beefy server.  Let people
do that with the usual J=nn option already supported by the other
docker-based tests.

Paolo

Paolo Bonzini (2):
  docker: allow customizing Travis global_env variables
  travis: move make -j flag out of script

 .travis.yml            | 3 ++-
 tests/docker/travis.py | 6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

-- 
2.13.0


Re: [Qemu-devel] [PATCH 0/2] docker: allow running "docker-travis" with arbitrary parallelism
Posted by Alex Bennée 6 years, 9 months ago
Paolo Bonzini <pbonzini@redhat.com> writes:

> docker-travis is limited to the -j3 flag specified for running on the
> SaaS Travis CI infrastructure.
>
> When running the test matrix locally with docker, however, we can run
> much more jobs at the same time on a suitably beefy server.  Let people
> do that with the usual J=nn option already supported by the other
> docker-based tests.

Eh, I was debugging stuff on our beefy server with J=20 and then I
realised I was just running:

  make docker-test-quick@travis J=20

So I guess some people do use the pseudo Travis expansion then?

>
> Paolo
>
> Paolo Bonzini (2):
>   docker: allow customizing Travis global_env variables
>   travis: move make -j flag out of script
>
>  .travis.yml            | 3 ++-
>  tests/docker/travis.py | 6 +++---
>  2 files changed, 5 insertions(+), 4 deletions(-)


--
Alex Bennée

Re: [Qemu-devel] [PATCH 0/2] docker: allow running "docker-travis" with arbitrary parallelism
Posted by Paolo Bonzini 6 years, 9 months ago
On 14/07/2017 12:05, Alex Bennée wrote:
> 
> Paolo Bonzini <pbonzini@redhat.com> writes:
> 
>> docker-travis is limited to the -j3 flag specified for running on the
>> SaaS Travis CI infrastructure.
>>
>> When running the test matrix locally with docker, however, we can run
>> much more jobs at the same time on a suitably beefy server.  Let people
>> do that with the usual J=nn option already supported by the other
>> docker-based tests.
> 
> Eh, I was debugging stuff on our beefy server with J=20 and then I
> realised I was just running:
> 
>   make docker-test-quick@travis J=20
> 
> So I guess some people do use the pseudo Travis expansion then?

How would that end up running tests/docker/travis?

Paolo

>>
>> Paolo
>>
>> Paolo Bonzini (2):
>>   docker: allow customizing Travis global_env variables
>>   travis: move make -j flag out of script
>>
>>  .travis.yml            | 3 ++-
>>  tests/docker/travis.py | 6 +++---
>>  2 files changed, 5 insertions(+), 4 deletions(-)
> 
> 
> --
> Alex Bennée
> 


Re: [Qemu-devel] [PATCH 0/2] docker: allow running "docker-travis" with arbitrary parallelism
Posted by Alex Bennée 6 years, 9 months ago
Paolo Bonzini <pbonzini@redhat.com> writes:

> On 14/07/2017 12:05, Alex Bennée wrote:
>>
>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>
>>> docker-travis is limited to the -j3 flag specified for running on the
>>> SaaS Travis CI infrastructure.
>>>
>>> When running the test matrix locally with docker, however, we can run
>>> much more jobs at the same time on a suitably beefy server.  Let people
>>> do that with the usual J=nn option already supported by the other
>>> docker-based tests.
>>
>> Eh, I was debugging stuff on our beefy server with J=20 and then I
>> realised I was just running:
>>
>>   make docker-test-quick@travis J=20
>>
>> So I guess some people do use the pseudo Travis expansion then?
>
> How would that end up running tests/docker/travis?

It runs the normal build and make check but in the Travis build
environment. Essentially the .travis.yml is just a bunch of alternative
configure invocations, there is nothing to stop you manually setting up
a build with TARGET_LIST and EXTRA_CONFIGURE_OPTS which is what I do if
there is a particular Travis test which is failing.

>
> Paolo
>
>>>
>>> Paolo
>>>
>>> Paolo Bonzini (2):
>>>   docker: allow customizing Travis global_env variables
>>>   travis: move make -j flag out of script
>>>
>>>  .travis.yml            | 3 ++-
>>>  tests/docker/travis.py | 6 +++---
>>>  2 files changed, 5 insertions(+), 4 deletions(-)
>>
>>
>> --
>> Alex Bennée
>>


--
Alex Bennée

Re: [Qemu-devel] [PATCH 0/2] docker: allow running "docker-travis" with arbitrary parallelism
Posted by Paolo Bonzini 6 years, 9 months ago
On 14/07/2017 15:56, Alex Bennée wrote:
> 
> Paolo Bonzini <pbonzini@redhat.com> writes:
> 
>> On 14/07/2017 12:05, Alex Bennée wrote:
>>>
>>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>>
>>>> docker-travis is limited to the -j3 flag specified for running on the
>>>> SaaS Travis CI infrastructure.
>>>>
>>>> When running the test matrix locally with docker, however, we can run
>>>> much more jobs at the same time on a suitably beefy server.  Let people
>>>> do that with the usual J=nn option already supported by the other
>>>> docker-based tests.
>>>
>>> Eh, I was debugging stuff on our beefy server with J=20 and then I
>>> realised I was just running:
>>>
>>>   make docker-test-quick@travis J=20
>>>
>>> So I guess some people do use the pseudo Travis expansion then?
>>
>> How would that end up running tests/docker/travis?
> 
> It runs the normal build and make check but in the Travis build
> environment. Essentially the .travis.yml is just a bunch of alternative
> configure invocations, there is nothing to stop you manually setting up
> a build with TARGET_LIST and EXTRA_CONFIGURE_OPTS which is what I do if
> there is a particular Travis test which is failing.

This is different, I want to run all Travis builds to ensure I don't
break one before going on vacation. :)

Paolo