Re: [Qemu-devel] [PULL 6/6] travis: Make check-acceptance job more verbose

[Philippe Mathieu-Daudé]

On 6/12/19 10:52 PM, Eduardo Habkost wrote:
> On Wed, Jun 12, 2019 at 10:49:18PM +0200, Philippe Mathieu-Daudé wrote:
>> On 6/11/19 7:14 PM, Eduardo Habkost wrote:
>>> It will help us debug issues when tests fail.
>>>
>>> Signed-off-by: Eduardo Habkost <address@hidden>
>>> ---
>>>  .travis.yml | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/.travis.yml b/.travis.yml
>>> index b053a836a3..a08a7b7278 100644
>>> --- a/.travis.yml
>>> +++ b/.travis.yml
>>> @@ -225,7 +225,7 @@ matrix:
>>>      # Acceptance (Functional) tests
>>>      - env:
>>>          - CONFIG="--python=/usr/bin/python3 
>>> --target-list=x86_64-softmmu,mips-softmmu,mips64el-softmmu,aarch64-softmmu,arm-softmmu,s390x-softmmu,alpha-softmmu"
>>> -        - TEST_CMD="make check-acceptance"
>>> +        - TEST_CMD="make AVOCADO_SHOW=test check-acceptance"
>>>        addons:
>>>          apt:
>>>            packages:
>>>
>>
>> Cleber, can you help figuring what's wrong here?
>>
>> https://travis-ci.org/qemu/qemu/jobs/544805900
>>
>> The tests seems successfull, is it possible the verbose logging returns
>> error to the shell even if the test succeed?
> 
> That's exactly what this patch does.  On my next pull request I
> will replace this with Cleber's patch that dumps the error log
> only if tests failed.

Ah OK, this is annoying because now all Travis builds are marked as
failed. Maybe Peter can apply your patch directly to master as a build fix?