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.

Signed-off-by: Anas Nashif <[email protected]>
  • Loading branch information
nashif committed Jun 8, 2023
1 parent c1e40bb commit 7f0aff9
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 7f0aff9

Please sign in to comment.