Re: [PATCH] tests/avocado/tuxrun_baselines: Fix ppc64 tests for binaries without slirp

[Thomas Huth]

On 07/06/2023 06.22, Joel Stanley wrote:
> On Tue, 6 Jun 2023 at 19:28, Thomas Huth <thuth@redhat.com> wrote:
> > The ppc64 tuxrun tests are currently failing if "slirp" has been
> > disabled in the binary since they are using "-netdev user" now.
> > We have to skip the test if this network backend is missing.
>
> Do the boot tests require networking? I doubt they do.
>
> You could instead remove the -netdev user option if slirp is not present.

Yes, that's an option. Not sure whether it's worth the effort here, though, 
since most of our CI pipelines run with slirp enabled anyway (it's just one 
job, avocado-cfi-ppc64-s390x, which is failing due to this missing check)

 Thomas


> > Fixes: 6ee3624236 ("improve code coverage for ppc64")
> > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > ---
> >   tests/avocado/tuxrun_baselines.py | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/avocado/tuxrun_baselines.py 
> > b/tests/avocado/tuxrun_baselines.py
> > index 3a46e7a745..e12250eabb 100644
> > --- a/tests/avocado/tuxrun_baselines.py
> > +++ b/tests/avocado/tuxrun_baselines.py
> > @@ -184,6 +184,7 @@ def common_tuxrun(self,
> >
> >       def ppc64_common_tuxrun(self, sums, prefix):
> >           # add device args to command line.
> > +        self.require_netdev('user')
> >           self.vm.add_args('-netdev', 'user,id=vnet,hostfwd=:127.0.0.1:0-:22',
> >                            '-device', 'virtio-net,netdev=vnet')
> >           self.vm.add_args('-netdev', '{"type":"user","id":"hostnet0"}',
> > --
> > 2.31.1