I am encountering an issue when I am trying to use split-synchronizer, which uses go-split-common library.
In my environment, Redis is deployed as cluster mode and sentinel mode is enabled with sentinel password.
When I passed the Redis settings to split-synchronizer, I got "Unauthenticated" error from Redis. After troubleshooting, I found it caused by incorrect sentinel password.
Currently go-split-common doesn't support sentinel password, while the dependency package splitio/go-toolkit supports this setting (https://github.com/splitio/go-toolkit/blob/main/redis/types.go#L25).
So could you please try to add this SentinelPassword parameter support?
As my investigation, just adding two lines in 2 files respectively is enough:
- Add new code
SentinelPassword string after:
- Add new code
universalOptions.SentinelPassword = config.SentinelPassword after:
|
universalOptions.Addrs = config.SentinelAddresses |
I am encountering an issue when I am trying to use split-synchronizer, which uses go-split-common library.
In my environment, Redis is deployed as cluster mode and sentinel mode is enabled with sentinel password.
When I passed the Redis settings to split-synchronizer, I got "Unauthenticated" error from Redis. After troubleshooting, I found it caused by incorrect sentinel password.
Currently go-split-common doesn't support sentinel password, while the dependency package splitio/go-toolkit supports this setting (https://github.com/splitio/go-toolkit/blob/main/redis/types.go#L25).
So could you please try to add this
SentinelPasswordparameter support?As my investigation, just adding two lines in 2 files respectively is enough:
SentinelPassword stringafter:go-split-commons/conf/conf.go
Line 46 in 82c4b0f
universalOptions.SentinelPassword = config.SentinelPasswordafter:go-split-commons/storage/redis/redis.go
Line 48 in 82c4b0f