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

docs: deprecate redis-based caching #9441

Merged
merged 9 commits into from
Sep 27, 2024
Merged

docs: deprecate redis-based caching #9441

merged 9 commits into from
Sep 27, 2024

Conversation

tacole02
Copy link
Contributor

What this PR does

This PR marks the experimental support for Redis-based caching as deprecated.

Which issue(s) this PR fixes or relates to

Fixes https://github.com/grafana/mimir-squad/issues/2414

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.

@tacole02 tacole02 requested a review from a team as a code owner September 26, 2024 23:36
@tacole02 tacole02 self-assigned this Sep 26, 2024
@tacole02 tacole02 added the type/docs Improvements or additions to documentation label Sep 26, 2024
Comment on lines 4828 to 4831
{{< admonition type="caution" >}}
Starting with Mimir 2.14, Redis-based caching is deprecated.
It will be removed in a future release.
{{< /admonition >}}
Copy link
Contributor

@narqo narqo Sep 27, 2024

Choose a reason for hiding this comment

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

Update discussed with @56quarters, and it seems that we don't plan to touch anything in the code. Deprecating with only mentioning it in the release notes, and the docs, works for me. We can do that in 2.14 👍


2.14 is planned to be release in 1+ week, with the rc.0 today. I'm not sure, I would rush in marking everything about redis in the code in the release branch as deprecated to support this statement. Can we deprecate it in 2.15?

Copy link
Contributor

Choose a reason for hiding this comment

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

We want to deprecated it in 2.14 to give people as much time to migrate as possible.

56quarters added a commit that referenced this pull request Sep 27, 2024
Part of grafana/mimir-squad#2414

Related #9441

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
56quarters added a commit that referenced this pull request Sep 27, 2024
Part of grafana/mimir-squad#2414

Related #9441

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
56quarters added a commit that referenced this pull request Sep 27, 2024
Part of grafana/mimir-squad#2414

Related #9441

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
56quarters added a commit that referenced this pull request Sep 27, 2024
* Mark Redis cache backend options as deprecated

Part of grafana/mimir-squad#2414

Related #9441

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

* Update CHANGELOG.md

---------

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
narqo pushed a commit that referenced this pull request Sep 27, 2024
* Mark Redis cache backend options as deprecated
* Update CHANGELOG.md

Part of grafana/mimir-squad#2414

Related #9441

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

---------

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
(cherry picked from commit d435e78)
Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>
@56quarters 56quarters merged commit 6c31ff7 into main Sep 27, 2024
29 checks passed
@56quarters 56quarters deleted the docs/ms-2414 branch September 27, 2024 18:10
narqo pushed a commit that referenced this pull request Oct 9, 2024
* Mark Redis cache backend options as deprecated
* Update CHANGELOG.md

Part of grafana/mimir-squad#2414

Related #9441

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

---------

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
(cherry picked from commit d435e78)
Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>
narqo added a commit that referenced this pull request Oct 9, 2024
* Mark Redis cache backend options as deprecated (#9453)

* Mark Redis cache backend options as deprecated
* Update CHANGELOG.md

Part of grafana/mimir-squad#2414

Related #9441

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

---------

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
(cherry picked from commit d435e78)
Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

* update version

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

* fixup! update version

* update changelog

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

---------

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>
Co-authored-by: Nick Pillitteri <56quarters@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants