Provide support for scaling based on Redis Key-Values #1493
Conversation
Signed-off-by: wangrushen <[email protected]>
|
Thank you for your contribution! 🙏 We will review your PR as soon as possible.
Learn more about:
|
❌ Deploy Preview for keda failed.
|
| @@ -1,15 +1,15 @@ | |||
| +++ | |||
| title = "Redis Lists" | |||
| title = "Redis Lists Or Strings" | |||
There was a problem hiding this comment.
I renamed redis-lists.md to redis.md because there is more than just a list, is this acceptable?
There was a problem hiding this comment.
I think that this could break the current naming of the urls. @zroubalik @wozniakjan ?
There was a problem hiding this comment.
in that case, we should probably also rename all the references, no?
not sure how many of these are relevant for a rename
$ grep . -nre '/scalers/redis-lists' | wc -l
1448
There was a problem hiding this comment.
If change, then I would probably go with just Redis
Do you mean breaking urls between versions of this scaler, or what? Then we can probably try to find a solution or just keep the original name.
| @@ -1,15 +1,15 @@ | |||
| +++ | |||
| title = "Redis Lists" | |||
| title = "Redis Lists Or Strings" | |||
There was a problem hiding this comment.
I think that this could break the current naming of the urls. @zroubalik @wozniakjan ?
Signed-off-by: wangrushen <[email protected]>
Provide support for scaling based on Redis Key-Values
Checklist
Related kedacore/keda#5003 kedacore/keda#6276