ddev / ddev-platformsh

Add integration with Platform.sh hosting service
Apache License 2.0
9 stars 10 forks source link

Support redis 7.0 + 6.2 #104

Open ptmkenny opened 1 year ago

ptmkenny commented 1 year ago

PlatformSH now supports redis 7.0 and 6.2, and has marked 6.0 and below as deprecated.

It appears that this addon has redis hard-coded to 6.0: https://github.com/ddev/ddev-platformsh/blob/main/platformsh/generate_redis_relationship.sh

It would be great if the version could be read from platform's services.yaml and set up accordingly.

rfay commented 1 year ago

Actually, you're not correct. The redis add-on for ddev has

    image: redis:6

https://github.com/ddev/ddev-redis/blob/9aff5269f5d46985d5a4f8030965808853a9847c/docker-compose.redis.yaml#L6

That will get 6.2, see https://hub.docker.com/_/redis

But the basic issue is that ddev get doesn't provide any way to specify arguments, hopefully it will

This is

rfay commented 1 year ago

Also

will impact this

rfay commented 10 months ago

redis 7 is being worked on actively in

There's also now a redis-7 add-on, https://github.com/oblakstudio/ddev-redis-7