Re: [PATCH] Increase timeout for tests and syntax-check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 29, 2021 at 04:20:53PM +0100, Andrea Bolognani wrote:
> On Fri, 2021-01-29 at 14:53 +0000, Daniel P. Berrangé wrote:
> > On Fri, Jan 29, 2021 at 03:46:18PM +0100, Andrea Bolognani wrote:
> > > On Fri, 2021-01-29 at 12:48 +0000, Daniel P. Berrangé wrote:
> > > > I'm not convinced we want to optimize for the slowest hardware
> > > > we can find, especially when there's an easy option of setting
> > > > --timeout-multiplier=10.
> > > 
> > > It's not complicated to add the option, but the fact that Debian,
> > > SUSE and now Fedora all need to specify a timeout multiplier hints to
> > > the fact that perhaps the default timeout is just too small.
> > 
> > AFAIK, Fedora hasn't set any timeout multiplier in our builds.
> 
> With
> 
>   commit 70307548d1622bfeffd60c86e62098d165c95074
>   Author: Cole Robinson <crobinso@xxxxxxxxxx>
>   Date:   Thu Jan 21 16:54:45 2021 -0500
> 
>     spec: Increase meson test timeout 10x
> 
>     Tests time out when building in slow environments, like emulated
>     s390x in Fedora copr. Bump up the test timeout
> 
>     Reviewed-by: Neal Gompa <ngompa13@xxxxxxxxx>
>     Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>
>     Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
> 
> 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.

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 :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux