Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests/sys/timer_overhead: disable test on native in CI #20324

Merged
merged 1 commit into from
Feb 1, 2024
Merged

tests/sys/timer_overhead: disable test on native in CI #20324

merged 1 commit into from
Feb 1, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented Feb 1, 2024

Contribution description

This disables the tests/sys/timer_overhead test run on native in the CI. Citing the comment in the Makefile:

# The test is sensitive to background CPU load. On the CI workers a lot of
# compilation tasks are run in parallel, making this test randomly fail.

Testing procedure

Green CI

Issues/PRs references

None

@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Feb 1, 2024
@maribu maribu requested a review from benpicco February 1, 2024 08:57
@maribu maribu added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 1, 2024
@riot-ci
Copy link

riot-ci commented Feb 1, 2024

Murdock results

✔️ PASSED

01aa5d8 tests/sys/timer_overhead: disable test on native in CI

Success Failures Total Runtime
17 0 17 01m:02s

Artifacts

@benpicco benpicco requested a review from kaspar030 February 1, 2024 11:55
kaspar030
kaspar030 approved these changes Feb 1, 2024
View reviewed changes
Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK.

@kaspar030 kaspar030 enabled auto-merge February 1, 2024 12:09
@kaspar030 kaspar030 added this pull request to the merge queue Feb 1, 2024
Merged via the queue into RIOT-OS:master with commit e48cbb2 Feb 1, 2024
27 checks passed
@maribu maribu deleted the tests/sys/ztimer_overhead branch February 1, 2024 16:44
@maribu
Copy link
Member Author

maribu commented Feb 1, 2024

Thx

@MrKevinWeiss MrKevinWeiss mentioned this pull request Feb 22, 2024
Merged
2 tasks
@MrKevinWeiss MrKevinWeiss added this to the Release 2024.04 milestone Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaspar030 kaspar030 kaspar030 approved these changes

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@benpicco benpicco Awaiting requested review from benpicco

Assignees
No one assigned
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Milestone
Release 2024.04
Development

Successfully merging this pull request may close these issues.
4 participants
@maribu @riot-ci @kaspar030 @MrKevinWeiss