Support Redis Sentinel on Svix Server

[tu-pm]

Feature Request

Motivation

Redis Sentinel is a common HA practice for a small production deployment of Redis, since it requires much less physical resources to ensure high availability compare to Redis Cluster. It'd be nice if Svix supports Sentinel as well.

Proposal

Alternatives

[tasn]

@svix-james, what do you think?

[svix-james]

I know next to nothing about sentinel so am not positive offhand, but I'm sure it's doable. The library supports sentinel.

[andrewmatthews]

Hi guys, I'm wondering if there is any more information on this? If the library supports Sentinel, perhaps it is simply a matter of the correct configuration. This is how other Redis libraries work.

[sanjaywinjit]

How about support for AWS ElasticCache ?

maintainer note: the author created a separate issue at https://github.com/svix/svix-webhooks/issues/1458

[svix-james]

@andrewmatthews We now have support for Redis Sentinel in main. It should be available in our next release. In order to try it, you'll need to at a minimum add these configuration parameters.

redis_dsn -- should point to a sentinel server sentinel_service_name -- the name of the sentinel service

I don't recommend this for production workloads at this point without a good amount of testing. It has been lightly tested on our end, and there are some upstream optimizations that need to be made in the redis library. I will try to work on that at some point.

Fixed by https://github.com/svix/svix-webhooks/pull/1472