From 9b4260fc782d16e5c84221d8e11e3daa464b82dd Mon Sep 17 00:00:00 2001 From: Jeremy Bettis Date: Tue, 27 Sep 2022 15:33:48 -0600 Subject: cq: Only run twister once in test phase Instead of running twister --build-only in the build phase and twister --test-only in the test phase, just run a normal clean twister build once in the test phase. I don't know that it happens in the CQ, but when running firmware_builder.py locally, the --test-only step is often failing. BRANCH=None BUG=None TEST=Ran locally. Signed-off-by: Jeremy Bettis Change-Id: I43cf9f5c360cb7de43fa6ae59128948f15230092 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3922446 Tested-by: Jeremy Bettis Reviewed-by: Tristan Honscheid Auto-Submit: Jeremy Bettis Commit-Queue: Tristan Honscheid --- zephyr/firmware_builder.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/zephyr/firmware_builder.py b/zephyr/firmware_builder.py index 436760361a..14b1e06cb9 100755 --- a/zephyr/firmware_builder.py +++ b/zephyr/firmware_builder.py @@ -97,8 +97,6 @@ def build(opts): with open(opts.metrics, "w") as file: file.write(json_format.MessageToJson(metric_list)) - run_twister(platform_ec, opts.code_coverage, ["--build-only"]) - UNITS = { "B": 1, @@ -245,7 +243,7 @@ def test(opts): # Twister-based tests platform_ec = zephyr_dir.parent third_party = platform_ec.parent.parent / "third_party" - run_twister(platform_ec, opts.code_coverage, ["--test-only"]) + run_twister(platform_ec, opts.code_coverage) if opts.code_coverage: build_dir = platform_ec / "build" / "zephyr" -- cgit v1.2.1