On Mon, May 05, 2025 at 10:46:52AM -0700, Pierrick Bouvier wrote:
> On 5/5/25 3:32 AM, Thomas Huth wrote:
> > On 03/05/2025 22.18, Pierrick Bouvier wrote:
> > > Allows all tests to be visible by default when using meson test
> > > directly.
> > >
> > > This has no impact on make check-* commands, which use SPEED=quick by
> > > default (see scripts/mtest2make.py).
> > >
> > > Signed-off-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>
> > > ---
> > > meson.build | 5 +++--
> > > 1 file changed, 3 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/meson.build b/meson.build
> > > index f8bf6e7bb66..57ff3f722d8 100644
> > > --- a/meson.build
> > > +++ b/meson.build
> > > @@ -5,12 +5,13 @@ project('qemu', ['c'], meson_version: '>=1.5.0',
> > > meson.add_devenv({ 'MESON_BUILD_ROOT' : meson.project_build_root() })
> > > -add_test_setup('quick', exclude_suites: ['slow', 'thorough'],
> > > is_default: true,
> > > +add_test_setup('quick', exclude_suites: ['slow', 'thorough'],
> > > env: ['RUST_BACKTRACE=1'])
> > > add_test_setup('slow', exclude_suites: ['thorough'],
> > > env: ['G_TEST_SLOW=1', 'SPEED=slow',
> > > 'RUST_BACKTRACE=1'])
> > > add_test_setup('thorough',
> > > - env: ['G_TEST_SLOW=1', 'SPEED=thorough',
> > > 'RUST_BACKTRACE=1'])
> > > + env: ['G_TEST_SLOW=1', 'SPEED=thorough',
> > > 'RUST_BACKTRACE=1'],
> > > + is_default: true)
> >
> > I'd rather not make it the default: The thorough functional tests download a
> > lot of assets from the internet, so if someone just runs "meson test"
> > without further parameters, I think we should not trigger these downloads in
> > that case.
> >
>
> $ cat tests/Makefile.include
> ...
> check-functional:
> @$(NINJA) precache-functional
> @QEMU_TEST_NO_DOWNLOAD=1 $(MAKE) SPEED=thorough check-func
> ...
>
> What's the rationale to run check-func with SPEED=thorough and
> QEMU_TEST_NO_DOWNLOAD=1 with precache-functional having cached only quick
> tests by default?
Are you sure about that ? The precache logic uses custom_target() in
meson and so does not filters on "suites" used to define quick vs
thorough tests.
IOW, precache should always be downloading all assets.
> Either we should remove SPEED=thorough, or download all tests by default.
>
> > Thomas
> >
>
With 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 :|
