Re: [PATCH 3/3] tests/avocado: Enable reverse_debugging.py tests in gitlab CI

[Thomas Huth]

On 21/11/2023 10.14, Daniel P. Berrangé wrote:
> On Tue, Nov 21, 2023 at 09:56:24AM +0100, Thomas Huth wrote:
> > On 17/11/2023 08.35, Nicholas Piggin wrote:
> > > On Fri Nov 17, 2023 at 4:11 AM AEST, Thomas Huth wrote:
> > > > On 16/11/2023 12.53, Nicholas Piggin wrote:
> > > > > Let's try enable reverse_debugging.py in gitlab CI.
> > > > >
> > > > > Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
> > > > > ---
> > > > > Maybe we could try this again at some point? The bug might have been
> > > > > noticed sooner.
> > > > >
> > > > > They only take a couple of seconds to run so should not take too much
> > > > > overhead. But my gitlab CI pipeline doesn't run the avocado tests for
> > > > > some reason, so I can't see if it's still causing problems.
...
> > > > FYI, I gave it a try, and it survived my CI run:
> > > >
> > > >     https://gitlab.com/thuth/qemu/-/jobs/5552213972#L403
> > > >
> > > > So I went ahead and put it (together with the first patch) in my current
> > > > pull request, let's see how it goes...
> > >
> > > Great, thank you.
> >
> > ... and here it's failing again (current master branch):
> >
> > https://gitlab.com/thuth/qemu/-/jobs/5582657378#L404
> >
> > According to the debug.log in the artifacts, it's failing here:
...
> 08:28:32 ERROR| Task.Reader: BrokenPipeError: [Errno 32] Broken pipe
>
> With seeing a bad packet from GDB and seeing Broken pipe from QMP,
> my impression is that the QEMU process is no longer present, most
> likely it has SEGV'd I reckon.
>
> IOW, I think we might well have a genuine bug here, not merely an
> unreliable test suite.
>
> None the less, unless someone can guess what the problem is, we'll
> need to disable the test to get reliable CI.

I'll sent a patch to revert the commit.

> A bug should be opened though with the CI logs.

Done: https://gitlab.com/qemu-project/qemu/-/issues/1992

 Thomas