Skip to content

Conversation

@JordanYates
Copy link
Contributor

@JordanYates JordanYates commented Dec 6, 2025

Display how long it takes to build the testplan in the logs. This can be useful since the duration to build the plan can vary wildly depending on the arguments given. Without this information, it is difficult to tell (from CI logs) whether a long execution time is from actually building and running the tests, or just generating the testplan.

For example, in a downstream repo:

./zephyr/scripts/twister --integration -iv -T . --dry-run

INFO    - Building initial testsuite list...
INFO    - Built testsuite list in 10.58 seconds

vs

./zephyr/scripts/twister --integration -iv -T . --dry-run --vendor nordic --vendor zephyr

INFO    - Building initial testsuite list...
INFO    - Built testsuite list in 201.01 seconds

Display how long it takes to build the testplan in the logs. This can
be useful since the duration to bulid the plan can vary wildly depending
on the arguments given. Without this information, it is difficult to
tell (from CI logs) whether a long execution time is from actually
building and running the tests, or just generating the testplan.

For example, in a downstream repo:
```
./zephyr/scripts/twister --integration -iv -T . --dry-run

INFO    - Building initial testsuite list...
INFO    - Built testsuite list in 10.58 seconds
```
vs
```
./zephyr/scripts/twister --integration -iv -T . --dry-run \
--vendor nordic --vendor zephyr

INFO    - Building initial testsuite list...
INFO    - Built testsuite list in 201.01 seconds
```

Signed-off-by: Jordan Yates <[email protected]>
Display the total run duration to two decimal points instead of ~14,
similar to the other displayed timestamps.

Signed-off-by: Jordan Yates <[email protected]>
@JordanYates JordanYates added area: Twister Twister Enhancement Changes/Updates/Additions to existing features labels Dec 6, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 6, 2025

@JordanYates
Copy link
Contributor Author

CI failure unrelated, errors like: /home/runner/work/zephyr/zephyr/zephyr-sdk/arc64-zephyr-elf/bin/arc64-zephyr-elf-ar: unable to copy file 'zephyr/libzephyr.a'; reason: No space left on device

@nashif
Copy link
Member

nashif commented Dec 6, 2025

hotfix here #100616

@nashif nashif added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Dec 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Twister Twister Enhancement Changes/Updates/Additions to existing features Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants