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

[Eduardo Habkost]

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.

-- 
Eduardo