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

feat: set admin load balancer healthcheck timeout to 30 seconds #55

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

niross
Copy link
Contributor

@niross niross commented Feb 26, 2024

Heavy load on the datasets db can slow down responses from the /healthcheck endpoint. This in turn can cause ECS to start some replacement tasks and casus data workspace to start flapping.

Until we can resolve this in a nicer way, set the timeout to 30 seconds.

@niross niross requested a review from a team as a code owner February 26, 2024 16:05
@niross niross force-pushed the feat/bump-admin-healthcheck-timeout branch from 066249c to 02c5056 Compare February 26, 2024 16:12
Heavy load on the datasets db can slow down responses from
the /healthcheck endpoint. This in turn can cause ECS to
start some replacement tasks and casus data workspace to
start flapping.

Until we can resolve this in a nicer way, set the timeout to
30 seconds.
@niross niross force-pushed the feat/bump-admin-healthcheck-timeout branch from 02c5056 to 91d5175 Compare February 26, 2024 16:25
@niross niross merged commit eb86e04 into main Feb 27, 2024
1 check passed
@niross niross deleted the feat/bump-admin-healthcheck-timeout branch February 27, 2024 09:53
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