From: Otavio Salvador <otavio@ossystems.com.br>
To: openembedded-core@lists.openembedded.org
Cc: Paul Barker <paul@pbarker.dev>,
Otavio Salvador <otavio@ossystems.com.br>,
Luciano Dittgen <luciano.dittgen@ossystems.com.br>
Subject: [PATCH v2 1/3] yocto-check-layer: Add --run-tests to select the tests to run
Date: Mon, 19 Oct 2026 11:39:22 -0300 [thread overview]
Message-ID: <20261019143924.1521981-2-otavio@ossystems.com.br> (raw)
In-Reply-To: <20261019143924.1521981-1-otavio@ossystems.com.br>
The common and distro tests ignore --machines, so a BSP layer checked in
several runs, one per group of machines, repeats them in each. Add
--run-tests, taking <module>[.<class>[.<name>]] as oe-test does, so the
common tests can run once and only the BSP ones in the others. A name
matching no test is an error rather than a run that checked nothing.
Tested on meta-freescale with "--run-tests common distro" and
"--run-tests bsp --machines imx8mm-ddr4-evk imx8mn-ddr4-evk".
Suggested-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
AI-Generated: Uses Claude Code (Claude Opus 5.5)
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
scripts/yocto-check-layer | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/scripts/yocto-check-layer b/scripts/yocto-check-layer
index 8c8b4dbcad..e747451366 100755
--- a/scripts/yocto-check-layer
+++ b/scripts/yocto-check-layer
@@ -32,14 +32,14 @@ CASES_PATHS = [os.path.join(os.path.abspath(os.path.dirname(__file__)),
'lib', 'checklayer', 'cases')]
logger = scriptutils.logger_create(PROGNAME, stream=sys.stdout)
-def test_layer(td, layer, test_software_layer_signatures):
+def test_layer(td, layer, test_software_layer_signatures, run_tests):
from checklayer.context import CheckLayerTestContext
logger.info("Starting to analyze: %s" % layer['name'])
logger.info('Distro: %s' % td['bbvars']["DISTRO"])
logger.info("----------------------------------------------------------------------")
tc = CheckLayerTestContext(td=td, logger=logger, layer=layer, test_software_layer_signatures=test_software_layer_signatures)
- tc.loadTests(CASES_PATHS)
+ tc.loadTests(CASES_PATHS, modules=run_tests, modules_required=run_tests)
return tc.runTests()
def dump_layer_debug(layer):
@@ -70,6 +70,8 @@ def main():
help='check that software layers do not change signatures (on by default)')
group.add_argument('--without-software-layer-signature-check', action='store_false', dest='test_software_layer_signatures',
help='disable signature checking for software layers')
+ parser.add_argument('--run-tests', nargs='+', metavar='TEST', default=[],
+ help='Tests to run, as <module>[.<class>[.<name>]] (default: all)')
parser.add_argument('-n', '--no-auto', help='Disable auto layer discovery',
action='store_true')
parser.add_argument('-d', '--debug', help='Enable debug output',
@@ -228,7 +230,7 @@ def main():
layers_tested = layers_tested + 1
continue
- result = test_layer(td, layer, args.test_software_layer_signatures)
+ result = test_layer(td, layer, args.test_software_layer_signatures, args.run_tests)
results[layer['name']] = result
results_status[layer['name']] = 'PASS' if results[layer['name']].wasSuccessful() else 'FAIL'
layers_tested = layers_tested + 1
--
2.54.0
next prev parent reply other threads:[~2026-10-19 14:39 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-17 22:56 [PATCH 0/3] yocto-check-layer: Faster BSP checks over many machines Otavio Salvador
2026-10-17 22:56 ` [PATCH 1/3] yocto-check-layer: Add --run-tests to select the tests to run Otavio Salvador
2026-10-19 8:19 ` Paul Barker
2026-10-17 22:56 ` [PATCH 2/3] yocto-check-layer: Skip initial signatures unless test_signatures runs Otavio Salvador
2026-10-17 22:56 ` [PATCH 3/3] checklayer: Reuse signatures of an identical successful world dump Otavio Salvador
2026-10-19 9:04 ` Paul Barker
2026-10-19 10:01 ` Paul Barker
2026-10-19 14:39 ` [PATCH v2 0/3] yocto-check-layer: Faster BSP checks over many machines Otavio Salvador
2026-10-19 14:39 ` Otavio Salvador [this message]
2026-10-19 14:39 ` [PATCH v2 2/3] yocto-check-layer: Skip initial signatures unless test_signatures runs Otavio Salvador
2026-10-19 14:39 ` [PATCH v2 3/3] checklayer: Share each world signature dump between the tests needing it Otavio Salvador
2026-10-29 12:20 ` [OE-core] " Otavio Salvador
2026-10-30 7:05 ` Mathieu Dubois-Briand
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=20261019143924.1521981-2-otavio@ossystems.com.br \
--to=otavio@ossystems.com.br \
--cc=luciano.dittgen@ossystems.com.br \
--cc=openembedded-core@lists.openembedded.org \
--cc=paul@pbarker.dev \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox