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

Fix API timeout workflow failures #139

Closed
wants to merge 4 commits into from
Closed

Conversation

gabrielfeo
Copy link
Member

Increase API read timeout for the following workflows:

Tested to fix the timeout failures in 3 of the workflows (though they now fail for cache misses):

@gabrielfeo gabrielfeo changed the title Fix workflows failing for API timeout error Fix API timeout workflow failures Feb 24, 2025
Copy link
Member

@ribafish ribafish left a comment

Choose a reason for hiding this comment

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

LGTM

@gabrielfeo
Copy link
Member Author

gabrielfeo commented Feb 25, 2025

I'm closing this PR as it's unlikely to be a fix.

As @erichaagdev showed during the team, read.timeout is already set to 61s for many of these workflows' jobs, so it's unlikely that the timeout increase to 30s is a fix (timeout is set to 61s implicitly because of this and this on jobs that downloaded BVS with downloadDevelopmentRelease: true since this commit).

There's an ongoing thread with the infrastructure team about ingest time issues affecting the Develocity instance. We can revisit this once that's addressed.

@gabrielfeo gabrielfeo closed this Feb 25, 2025
@gabrielfeo gabrielfeo deleted the gf/fix-timeout-errors branch February 25, 2025 18:19
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