relay icon indicating copy to clipboard operation
relay copied to clipboard

Redis sentinel support

Open doc-sheet opened this issue 2 years ago • 6 comments

Hello.

Since 0.23.1 redis-rs supports sentinel. Could you add it to relay please?

doc-sheet avatar Dec 19 '23 22:12 doc-sheet

Hey,

thanks for opening the issue!

Could you, please, provide more context for this feature request? Why do you need it? How that will help you with using the Relay? Any specific issue it will solve for you?

It will give us more information to consider before plan it out and putting the work on our roadmap.

olksdr avatar Dec 20 '23 13:12 olksdr

Sure. I want to setup failover for redis connection. Sentinel is way simpler than redis cluster and supports select db statement so i can point whole sentry stack to the same redis machines.

Other (python) services can be configured to use sentinel with a bit of code in config but relay is the one where i need to manually update redis server address when db instance crashes.

On the other hand redis-rs now have native support for sentinel so relay could be much more stable in that scenario.

doc-sheet avatar Dec 20 '23 19:12 doc-sheet

Thanks for the additional context! I will bring this up to the team for discussion and followup actions.

olksdr avatar Dec 21 '23 09:12 olksdr

Hi @doc-sheet, We can take this with very low priority but you are welcome to contribute and we'll be happy to review.

jernejstrasner avatar Jan 03 '24 10:01 jernejstrasner

Sadly I never touched rust. I'll better leave it to someone else. Thank you!

doc-sheet avatar Jan 03 '24 10:01 doc-sheet

Hi! Could you look at #4979?

doc-sheet avatar Jul 27 '25 12:07 doc-sheet