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

Add per-tenant max cache freshness to query-frontend #2609

Merged
merged 10 commits into from
May 25, 2020

Conversation

annanay25
Copy link
Contributor

@annanay25 annanay25 commented May 19, 2020

What this PR does:
Add frontend.per-user-max-cache-freshness flag to query-frontend, which allows users to configure per-tenant max-cache-freshness values.

  • By default the globally configured frontend.max-cache-freshness value will be applied and hence there will be no breaking changes.
  • If however, a per-tenant value is specified for a specific tenant, the globally configured value will be overridden.

Which issue(s) this PR fixes:
Fixes NA

Checklist

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

annanay25 added 3 commits May 19, 2020 20:18
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Signed-off-by: Annanay <annanayagarwal@gmail.com>
@pull-request-size pull-request-size bot added size/M and removed size/L labels May 21, 2020
docs/configuration/config-file-reference.md Outdated Show resolved Hide resolved
pkg/util/validation/limits.go Outdated Show resolved Hide resolved
pkg/querier/queryrange/results_cache.go Outdated Show resolved Hide resolved
annanay25 added 2 commits May 22, 2020 13:32
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Signed-off-by: Annanay <annanayagarwal@gmail.com>
@pull-request-size pull-request-size bot added size/L and removed size/M labels May 22, 2020
annanay25 added 2 commits May 22, 2020 16:09
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Copy link
Contributor

@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 addressing my feedback! LGTM (modulo a couple of nits)

pkg/querier/queryrange/results_cache.go Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Signed-off-by: Annanay <annanayagarwal@gmail.com>
Copy link
Contributor

@gouthamve gouthamve left a comment

Choose a reason for hiding this comment

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

LGTM with question.

docs/configuration/config-file-reference.md Show resolved Hide resolved
@gouthamve gouthamve merged commit 1c76409 into cortexproject:master May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants