-
Notifications
You must be signed in to change notification settings - Fork 20
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
DBAAS: support for user operations #654
Conversation
060fecb
to
da30717
Compare
da30717
to
105090e
Compare
Hello! Could someone provide some context about this release, the roadmap, or anything else relevant so @ridge can work on the changelog and community update? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would not add Redis in any way.
It's going to be deprecated by Aiven (last week's news) and we will need to remove it from our CLI as well, during first half (quarter?) of 2025.
If instead we consider that since the work has been already done, better deliver it, then i would remove the ACLs management for Redis, which is not fully supported.
Removed all redis support from this PR ( efdee4f ) |
Nothing super new @clovisgenevard , we are simply aligning the clients. |
redis is going to be replaced as a drop in by valkey under the same naming: Caching, so not sure we should avoid doing redis now. |
ac6cb06
to
3e80a06
Compare
For one thing, the commands will all change name. To my understanding it's NOT an in-place substitution, is a NEW service, and customer will need to adapt to new endpoints, new commands, new naming etc etc. Their services will not just renamed "Valkey" under the hood, they will be MIGRATED to Valkey. I would not encourage using redis when we are going to deprecate it by the end of the month: it will soon not be possible anymore to create a new redis services and i doubt we will have Valkey before Februray at soonest. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @tgrondier !
Description
This adds for the following API calls:
Checklist
(For exoscale contributors)
Testing