Skip to content

Allow longer time for low performance architectures #198

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

U2FsdGVkX1
Copy link

@U2FsdGVkX1 U2FsdGVkX1 commented Nov 27, 2024

I am a Fedora packager and recently observed that many tests were failing due to timeouts during the check stage. This issue is particularly noticeable on low-performance devices and current RISC-V hardware. The existing 2-second timeout is insufficient for these systems, leading to test failures.

To address this, I have adjusted the timeout duration from 2 seconds to 5 seconds. This change has been tested on the SG2042 (a RISC-V chip) and has shown to work well.

Copy link

cla-checker-service bot commented Nov 27, 2024

💚 CLA has been signed

@pquentin
Copy link
Member

pquentin commented Jan 6, 2025

I'm slightly reluctant to make CI 3 seconds slower. However, I can't merge this until you have signed the CLA, sorry.

@U2FsdGVkX1
Copy link
Author

I'm slightly reluctant to make CI 3 seconds slower. However, I can't merge this until you have signed the CLA, sorry.

Oh I forgot about that. Sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants