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.

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

tests/sys/psa_crypto: increase timeout #20612

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

mguetschow
Copy link
Contributor

Contribution description

This should hopefully fix the failing nightly builds since ad9d501

The test uses ed25519 in software and thereby takes some time to finish. Increasing the timeout to 20s fixed it locally with a samr21-xpro board.

Testing procedure

Look at the minimal diff, merge and then hope for the next nightly.

Issues/PRs references

thanks to @maribu for mentioning the failing builds

to account for slow software implementation
@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Apr 23, 2024
@maribu maribu enabled auto-merge April 23, 2024 10:51
@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 Apr 23, 2024
@riot-ci
Copy link

riot-ci commented Apr 23, 2024

Murdock results

✔️ PASSED

db12864 tests/sys/psa_crypto: increase timeout

Success Failures Total Runtime
196 0 196 02m:23s

Artifacts

@maribu maribu added this pull request to the merge queue Apr 23, 2024
Merged via the queue into RIOT-OS:master with commit 3ae63a1 Apr 23, 2024
27 checks passed
@mguetschow mguetschow deleted the psa-test-timeout branch April 23, 2024 12:47
@mguetschow mguetschow added this to the Release 2024.07 milestone Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

3 participants