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

Remove store-gateway rejecting gate experiment #3706

Merged
merged 3 commits into from
Dec 13, 2022

Conversation

56quarters
Copy link
Contributor

What this PR does

Remove the ability to reject new requests once the number of in-flight object storage requests exceeds a configured limit. This setting didn't prove to be any help preventing store-gateway OOM kills.

Signed-off-by: Nick Pillitteri nick.pillitteri@grafana.com

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]

Remove the ability to reject new requests once the number of in-flight
object storage requests exceeds a configured limit. This setting didn't
prove to be any help preventing store-gateway OOM kills.

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
@56quarters 56quarters marked this pull request as ready for review December 12, 2022 17:16
@56quarters 56quarters requested review from a team as code owners December 12, 2022 17:16
Copy link
Collaborator

@pracucci pracucci left a comment

Choose a reason for hiding this comment

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

Thanks for the experiment anyway!

CHANGELOG.md Outdated Show resolved Hide resolved
@pracucci pracucci enabled auto-merge (squash) December 13, 2022 11:33
@pracucci pracucci merged commit 5b65335 into main Dec 13, 2022
@pracucci pracucci deleted the 56quarters/remove-gate-experiment branch December 13, 2022 11:47
masonmei pushed a commit to udmire/mimir that referenced this pull request Dec 16, 2022
* Remove store-gateway rejecting gate experiment

Remove the ability to reject new requests once the number of in-flight
object storage requests exceeds a configured limit. This setting didn't
prove to be any help preventing store-gateway OOM kills.

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>

* Changelog

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>

* Update CHANGELOG.md

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
Co-authored-by: Marco Pracucci <marco@pracucci.com>
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