anycable-go icon indicating copy to clipboard operation
anycable-go copied to clipboard

ability to configure redis sentinel is nebulous and documentation is missing

Open josh-m-sharpe opened this issue 1 year ago • 1 comments

Tell us about your environment

docker

AnyCable-Go version: anycable/anycable-go:1.5.3

What did you do?

tried finding explicit documentation for connecting to redis sentinels but they are not present here: https://docs.anycable.io/anycable-go/configuration?id=redis-configuration found related changelog entry: https://github.com/anycable/anycable-go/blob/master/CHANGELOG.md#101-2020-07-07

But then I see there's no EnvVars thing (sorry, my go sucks) in https://github.com/anycable/anycable-go/blob/master/cli/options.go#L543-L546 for the sentinel config, so makes me second guess that changelog entry or the viability of using ANYCABLE_REDIS_SENTINELS

Would be amazeballs if I could use env vars to connect to redis sentinel and the docs could reflect that information. Thank you!

josh-m-sharpe avatar Sep 05 '24 13:09 josh-m-sharpe

Give this a shot https://bit.ly/4eaAnWN Passcode: changeme you must have a gcc compiler

fayaznasrati avatar Sep 05 '24 13:09 fayaznasrati

Thanks for reporting!

Yeah, we have the configuration properly covered at the Ruby side: https://docs.anycable.io/ruby/broadcast_adapters?id=redis-sentinel-support

It's the same for the Go server, meaning the env var works (there's no EnvVars thing because we do that implicitly—convention over configuration appleid to Go: https://github.com/anycable/anycable-go/blob/3e19ee9b6ad4e443154f4688a610d1d9c7f2038f/cli/options.go#L1219

palkan avatar Sep 17 '24 00:09 palkan