Skip to content

Commit

Permalink
ci: testplan: do not start more runners than needed
Browse files Browse the repository at this point in the history
We have been launching 10 runners up a certain number of tests, although
we only neeed half of that for some scenarios.
Too many runners started that have to execute just a small number of
tests wastes times on setup and blocks the queue. Just start the number
of nodes needed based on initial calculation.

(cherry picked from commit 7f0aff9)

Original-Signed-off-by: Anas Nashif <[email protected]>
GitOrigin-RevId: 7f0aff9
Change-Id: I7da738be6d0abaf79b28e117a85c00c243298043
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/zephyr/+/4603285
Tested-by: Al Semjonovs <[email protected]>
Commit-Queue: Al Semjonovs <[email protected]>
Tested-by: CopyBot Service Account <[email protected]>
Reviewed-by: Al Semjonovs <[email protected]>
  • Loading branch information
nashif authored and Chromeos LUCI committed Jun 9, 2023
1 parent 8cf989c commit 01da023
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions scripts/ci/test_plan.py
Original file line number Diff line number Diff line change
Expand Up @@ -359,12 +359,6 @@ def parse_args():
else:
nodes = round(total_tests / args.tests_per_builder)

if total_tests % args.tests_per_builder != total_tests:
nodes = nodes + 1

if args.default_matrix > nodes > 5:
nodes = args.default_matrix

tp.write(f"TWISTER_TESTS={total_tests}\n")
tp.write(f"TWISTER_NODES={nodes}\n")
tp.write(f"TWISTER_FULL={f.full_twister}\n")
Expand Down

0 comments on commit 01da023

Please sign in to comment.