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

Support for Redis v7 and newer #56

Open
4n4nd opened this issue Aug 17, 2022 · 3 comments
Open

Support for Redis v7 and newer #56

4n4nd opened this issue Aug 17, 2022 · 3 comments

Comments

@4n4nd
Copy link
Contributor

4n4nd commented Aug 17, 2022

Hey team 👋 I was trying to upgrade my cluster from Redis version 6.2.x to 7.0.x. But it seems like v7 is not supported by the operator yet. This is the error that I get:

E0817 14:03:24.673063       1 redisnode.go:247] readiness check failed, err:Readiness failed, cluster slots response err: response returned from Conn: unmarshaling message off Conn: cannot decode resp array into string

Do we have any plans on supporting Redis v7?

Thanks!

@cin
Copy link
Contributor

cin commented Aug 17, 2022

We currently don't have any use cases that require 7.x, so we haven't looked into it yet. I'm not sure how much effort it will take to support both 6.x and 7.x, but it's definitely something we should look into as there are some nice features in the new version. The only problem is that we have a lot of other issues on our plate at the moment and won't be able to dedicate time to this in the near term. This is something we can try to get in our Q4 agenda.

@4n4nd 4n4nd closed this as not planned Won't fix, can't repro, duplicate, stale Aug 23, 2022
@4n4nd 4n4nd reopened this Sep 12, 2022
@RobocashGroup
Copy link

Hello, team.

Do you have any updates about this issue?
We need support redis v7 in operator.

@cin
Copy link
Contributor

cin commented May 31, 2023

I don't have an update on this issue. It looks like someone submitted a PR for it, but I think there's going to be more to it than simply changing the version. This repo is also not being actively maintained at the moment. I'm currently maintaining a fork here. I hope to get changes merged back here periodically. I'll gladly accept a PR for this too :)

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 a pull request may close this issue.

3 participants