[PATCH] Increase timeout for tests and syntax-check
Daniel P. Berrangé
berrange at redhat.com
Fri Jan 29 16:03:12 UTC 2021
On Fri, Jan 29, 2021 at 04:56:07PM +0100, Andrea Bolognani wrote:
> On Fri, 2021-01-29 at 15:28 +0000, Daniel P. Berrangé wrote:
> > On Fri, Jan 29, 2021 at 04:20:53PM +0100, Andrea Bolognani wrote:
> > > the timeout multiplier has been added to the upstream spec file,
> > > which the Fedora spec file is based on, so while you're correct that
> > > Fedora is not yet using it, it's fair to assume it will soon make its
> > > way there for the benefit of e.g. the virt-preview COPR.
> >
> > I think this is the wrong solution. IMHO RPM should be making th
> > %meson_test macro include the timeout arg automatically when
> > running on a emulated environment. This would fix the problem for
> > all users of meson, without causing unecessarily long timeouts
> > for native builds.
>
> How likely is it that some broken test will take longer than 30
> seconds and less than 5 minutes to run on your average developer's
> laptop? My experience suggests tests either take way less than half a
> minute to complete, or go on spinning forever.
>
> So yeah, in the unlikely event that your changes have introduced an
> infinite wait in one of the tests, it will take you a couple more
> minutes to realize that; however, considering how fast the test suite
> is under meson you'd probably get suspicious way before actually
> hitting the timeout, and introducing such an issue is hopefully not a
> very frequent occurrence anyway.
>
> Note that emulated environments are not the only ones hitting this:
> native builds on slow architectures failing because of the timeout
> are the reason why I introduced the timeout multiplier in Debian, for
> example.
>
> Finally, not everyone uses RPMs, and even those who do might not want
> to use "rpmbuild" as a substitute for "meson test" - especially not
> on the kind of hardware where the default test timeout is a problem!
>
> Raising the timeout at the meson level makes it possible for people
> to just run "meson test" and be reasonably confident it will only
> fail if there's an actual bug in libvirt.
If it is such a broad problem then the default should be raised in
meson itself rather than individual apps using meson.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list
mailing list