Skip to content

feat: use redis:7 instead of redis:6-bullseye, move Redis config, for #17 #38

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

Merged
merged 1 commit into from
Apr 25, 2025

Conversation

stasadev
Copy link
Member

@stasadev stasadev commented Apr 25, 2025

The Issue

We used old redis:6-bullseye only to support older systems.

This change was done almost 2 years ago, it's time to update.

How This PR Solves The Issue

  • Uses redis:7 by default.
  • Moves Redis config to the same location as in ddev/ddev-redis-7
  • People still can use redis:6-bullseye if they set it explicitly.

Manual Testing Instructions

ddev add-on get https://github.com/ddev/ddev-redis/tarball/20250425_stasadev_redis_7
ddev restart

# check for redis 7
ddev redis-cli INFO

Automated Testing Overview

Release/Deployment Notes

@stasadev stasadev changed the title feat: use redis:7 instead of redis:6-bullseye, move Redis config feat: use redis:7 instead of redis:6-bullseye, move Redis config, for #17 Apr 25, 2025
@stasadev stasadev merged commit 5e03450 into main Apr 25, 2025
2 checks passed
@stasadev stasadev deleted the 20250425_stasadev_redis_7 branch April 25, 2025 12:12
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.

1 participant