summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Bettis <jbettis@google.com>2022-09-27 15:33:48 -0600
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2022-09-28 20:28:43 +0000
commit9b4260fc782d16e5c84221d8e11e3daa464b82dd (patch)
treed34ba28372926f3e51f417b3ca5818ff5084f42e
parent5c4cfb226932720387b79f21e36595a4a122542c (diff)
downloadchrome-ec-9b4260fc782d16e5c84221d8e11e3daa464b82dd.tar.gz
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 <jbettis@google.com> Change-Id: I43cf9f5c360cb7de43fa6ae59128948f15230092 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3922446 Tested-by: Jeremy Bettis <jbettis@chromium.org> Reviewed-by: Tristan Honscheid <honscheid@google.com> Auto-Submit: Jeremy Bettis <jbettis@chromium.org> Commit-Queue: Tristan Honscheid <honscheid@google.com>
-rwxr-xr-xzephyr/firmware_builder.py4
1 files changed, 1 insertions, 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"