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

Increase default timeout for index-queries cache requests #6786

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

56quarters
Copy link
Contributor

What this PR does

As use of the index-queries cache has increased in the store-gateway, we see more and more timeouts making requests to it. Timeouts making a request mean the connection is closed and reopened since it assumed to be "bad". This can cause a huge number of connections on nodes to end up in TIME_WAIT.

Increase the timeout used by the index-queries cache to 450ms from the default of 200ms in jsonnet and helm.

Which issue(s) this PR fixes or relates to

N/A

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

As use of the index-queries cache has increased in the store-gateway, we
see more and more timeouts making requests to it. Timeouts making a request
mean the connection is closed and reopened since it assumed to be "bad". This
can cause a huge number of connections on nodes to end up in `TIME_WAIT`.

Increase the timeout used by the index-queries cache to 450ms from the default
of 200ms in jsonnet and helm.

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
@56quarters 56quarters marked this pull request as ready for review November 30, 2023 18:33
@56quarters 56quarters requested a review from a team as a code owner November 30, 2023 18:33
@56quarters 56quarters merged commit 7d7920f into main Nov 30, 2023
30 checks passed
@56quarters 56quarters deleted the 56quarters/index-cache-timeout branch November 30, 2023 21:05
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