From: "Daniel P. Berrangé" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Hanna Reitz" <hreitz@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"John Snow" <jsnow@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
qemu-block@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>,
"Kevin Wolf" <kwolf@redhat.com>, "Thomas Huth" <thuth@redhat.com>,
"Cleber Rosa" <crosa@redhat.com>,
"Daniel P. Berrangé" <berrange@redhat.com>
Subject: [PATCH v2 04/13] tests: ensure all qcow2 I/O tests are able to be run via make
Date: Mon, 12 Jan 2026 20:40:17 +0000 [thread overview]
Message-ID: <20260112204026.710659-5-berrange@redhat.com> (raw)
In-Reply-To: <20260112204026.710659-1-berrange@redhat.com>
For block formats marked as 'quick', only tests in the 'auto' group are
added to the meson test suite.
The result of this is that qcow2 tests not in the 'auto' group cannot be
run at all, even if passing SPEED=slow or SPEED=thorough.
To fix this we need todo two passes over the I/O test list. First add
all tests from 'auto' group into the 'block' suite, so they are run by
default. Then on the second pass add any tests which were not in 'auto'
into the 'block-slow' suite, so they get run when SPEED=slow or
SPEED=thorough.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
tests/qemu-iotests/meson.build | 48 ++++++++++++++++++++++++++++++----
1 file changed, 43 insertions(+), 5 deletions(-)
diff --git a/tests/qemu-iotests/meson.build b/tests/qemu-iotests/meson.build
index bf588cc2c9..1a24d801a3 100644
--- a/tests/qemu-iotests/meson.build
+++ b/tests/qemu-iotests/meson.build
@@ -30,16 +30,48 @@ endforeach
qemu_iotests_check_cmd = files('check')
foreach format, speed: qemu_iotests_formats
+ # Formats tagged 'quick' get the subset of tests in the 'auto'
+ # group, run by default with 'make check' / 'make check-block'
+ seen = []
if speed == 'quick'
- suites = 'block'
+ args = ['-tap', '-' + format, '-g', 'auto']
+ suites = ['block']
+
+ rc = run_command(
+ [python, qemu_iotests_check_cmd] + args + ['-n'],
+ check: true,
+ )
+
+ foreach item: rc.stdout().strip().split()
+ seen += item
+ args = [qemu_iotests_check_cmd,
+ '-tap', '-' + format, item,
+ '--source-dir', meson.current_source_dir(),
+ '--build-dir', meson.current_build_dir()]
+ # Some individual tests take as long as 45 seconds
+ # Bump the timeout to 3 minutes for some headroom
+ # on slow machines to minimize spurious failures
+ test('io-' + format + '-' + item,
+ python,
+ args: args,
+ depends: qemu_iotests_binaries,
+ env: qemu_iotests_env,
+ protocol: 'tap',
+ timeout: 180,
+ suite: suites)
+ endforeach
+ endif
+
+ suites = []
+ # Any format tagged quick or slow also gets added to slow
+ # otherwise its tagged thorough
+ if speed != 'thorough'
+ suites += ['block-slow']
else
- suites = ['block-' + speed]
+ suites += ['block-thorough']
endif
args = ['-tap', '-' + format]
- if speed == 'quick'
- args += ['-g', 'auto']
- endif
rc = run_command(
[python, qemu_iotests_check_cmd] + args + ['-n'],
@@ -47,6 +79,12 @@ foreach format, speed: qemu_iotests_formats
)
foreach item: rc.stdout().strip().split()
+ # Skip any tests already added from the 'auto' group
+ # as they're run in the 'quick' suite already
+ if item in seen
+ continue
+ endif
+
args = [qemu_iotests_check_cmd,
'-tap', '-' + format, item,
'--source-dir', meson.current_source_dir(),
--
2.52.0
next prev parent reply other threads:[~2026-01-12 20:41 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-12 20:40 [PATCH v2 00/13] tests: do more testing of block drivers Daniel P. Berrangé
2026-01-12 20:40 ` [PATCH v2 01/13] Revert "python/aqmp: fix send_fd_scm for python 3.6.x" Daniel P. Berrangé
2026-01-13 7:17 ` Thomas Huth
2026-01-14 16:57 ` John Snow
2026-01-14 17:10 ` Daniel P. Berrangé
2026-01-12 20:40 ` [PATCH v2 02/13] tests: print reason when I/O test is skipped in TAP mode Daniel P. Berrangé
2026-01-13 7:19 ` Thomas Huth
2026-01-12 20:40 ` [PATCH v2 03/13] tests: remove redundant meson suite for iotests Daniel P. Berrangé
2026-01-13 8:05 ` Thomas Huth
2026-01-12 20:40 ` Daniel P. Berrangé [this message]
2026-01-13 11:03 ` [PATCH v2 04/13] tests: ensure all qcow2 I/O tests are able to be run via make Thomas Huth
2026-01-12 20:40 ` [PATCH v2 05/13] scripts/mtest2make: ensure output has stable sorting Daniel P. Berrangé
2026-01-13 11:04 ` Thomas Huth
2026-01-12 20:40 ` [PATCH v2 06/13] scripts/mtest2make: support optional tests grouping Daniel P. Berrangé
2026-01-12 20:40 ` [PATCH v2 07/13] tests: add a meson suite / make target per block I/O tests format Daniel P. Berrangé
2026-01-12 20:40 ` [PATCH v2 08/13] docs/devel/testing: expand documentation for 'make check-block' Daniel P. Berrangé
2026-01-13 12:48 ` Thomas Huth
2026-01-12 20:40 ` [PATCH v2 09/13] tests: add nbd and luks to the I/O test suites Daniel P. Berrangé
2026-01-13 12:52 ` Thomas Huth
2026-01-14 10:45 ` Daniel P. Berrangé
2026-01-12 20:40 ` [PATCH v2 10/13] tests: skip I/O test 128 on GitLab due to broken dmsetup Daniel P. Berrangé
2026-01-13 12:54 ` Thomas Huth
2026-01-12 20:40 ` [PATCH v2 11/13] tests: fix check for sudo access in LUKS I/O test Daniel P. Berrangé
2026-01-13 12:56 ` Thomas Huth
2026-01-12 20:40 ` [PATCH v2 12/13] tests: add QEMU_TEST_IO_SKIP for skipping I/O tests Daniel P. Berrangé
2026-01-13 12:59 ` Thomas Huth
2026-01-12 20:40 ` [PATCH v2 13/13] gitlab: add jobs for thorough block tests Daniel P. Berrangé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260112204026.710659-5-berrange@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=crosa@redhat.com \
--cc=hreitz@redhat.com \
--cc=jsnow@redhat.com \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.