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

Mark Redis cache backend options as deprecated #9453

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

56quarters
Copy link
Contributor

Which issue(s) this PR fixes or relates to

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

Related #9441

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.

@56quarters 56quarters force-pushed the 56quarters/deprecate-redis-options branch from e13da68 to f84ba38 Compare September 27, 2024 14:41
@56quarters 56quarters marked this pull request as ready for review September 27, 2024 14:49
@56quarters 56quarters requested review from tacole02 and a team as code owners September 27, 2024 14:49
Copy link
Contributor

@narqo narqo left a comment

Choose a reason for hiding this comment

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

🔥 Thank you for collecting the list. We can backport it to release-2.14 after.

(nit) Shall we changelog it?

@56quarters 56quarters changed the base branch from release-2.14 to main September 27, 2024 15:23
@56quarters
Copy link
Contributor Author

🔥 Thank you for collecting the list. We can backport it to release-2.14 after.

(nit) Shall we changelog it?

Sure, I'll add a changelog entry. This PR was originally opened against the release branch but I'll change to main and we can backport to the release branch.

Part of grafana/mimir-squad#2414

Related #9441

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
@56quarters 56quarters force-pushed the 56quarters/deprecate-redis-options branch from f84ba38 to 2bace0f Compare September 27, 2024 15:35
CHANGELOG.md Outdated Show resolved Hide resolved
@56quarters 56quarters merged commit d435e78 into main Sep 27, 2024
29 checks passed
@56quarters 56quarters deleted the 56quarters/deprecate-redis-options branch September 27, 2024 15:57
@grafanabot
Copy link
Contributor

The backport to release-2.14 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new branch
git switch --create backport-9453-to-release-2.14 origin/release-2.14
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x d435e78bf54c3ee12c14490334114cbc3ce4aaec
# Push it to GitHub
git push --set-upstream origin backport-9453-to-release-2.14
git switch main
# Remove the local backport branch
git branch -D backport-9453-to-release-2.14

Then, create a pull request where the base branch is release-2.14 and the compare/head branch is backport-9453-to-release-2.14.

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>
@narqo narqo mentioned this pull request Sep 27, 2024
4 tasks
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants