Skip to content

Commit

Permalink
Update sentinel conf access string to allow hello channel access (val…
Browse files Browse the repository at this point in the history
…key-io#854)

This example of a minimal user account in your Valkey server
for Sentinel is incorrect. If you add this ACL as-is to your
valkey users.acl, valkey will add resetchannels -@ALL before
the +client which prevents sentinel from publishing messages
to the __sentinel__:hello pubsub for sentinel discovery.

Fix valkey-io#744.

Signed-off-by: Harkrishn Patro <[email protected]>
  • Loading branch information
hpatro authored Aug 3, 2024
1 parent facd123 commit 0fc43ed
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion sentinel.conf
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ sentinel monitor mymaster 127.0.0.1 6379 2
# Sentinel instances, should be configured along the following lines:
#
# user sentinel-user >somepassword +client +subscribe +publish \
# +ping +info +multi +slaveof +config +client +exec on
# +ping +info +multi +slaveof +config +client +exec &__sentinel__:hello on

# sentinel down-after-milliseconds <master-name> <milliseconds>
#
Expand Down

0 comments on commit 0fc43ed

Please sign in to comment.