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

RSDK-6097 increase analyzetests connection timeout #225

Merged
merged 1 commit into from
Dec 15, 2023
Merged

RSDK-6097 increase analyzetests connection timeout #225

merged 1 commit into from
Dec 15, 2023

Conversation

abe-winter
Copy link
Member

What changed

  • increase connection timeout from 5 seconds to 10 seconds

Why

Intermittent failures like this one:

https://github.com/viamrobotics/app/actions/runs/7024504474/job/19113759522?pr=3012#step:6:7464

2023-11-28T21:20:07.524Z FATAL analyzetests [email protected]/runtime.go:78 timed out while checking out a connection from connection pool: context deadline exceeded; maxPoolSize: 100, connections in use by cursors: 0, connections in use by transactions: 0, connections in use by other operations: 1

@abe-winter abe-winter requested a review from dgottlieb December 15, 2023 16:39
@viambot viambot added the safe to test Mark as safe to test label Dec 15, 2023
@abe-winter abe-winter merged commit fb6f67c into main Dec 15, 2023
7 checks passed
@abe-winter abe-winter deleted the analyze-test-timeout branch December 15, 2023 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgottlieb dgottlieb dgottlieb approved these changes

Assignees
No one assigned
Labels
safe to test Mark as safe to test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abe-winter @dgottlieb @viambot