bloomberg / redis-cookbook

A set of Chef recipes for installing and configuring Redis.
Apache License 2.0
19 stars 19 forks source link

Support sentinel announce-ip #17

Open richmolj opened 7 years ago

richmolj commented 7 years ago

From sentinel.conf:

# sentinel announce-ip <ip>
# sentinel announce-port <port>
#
# The above two configuration directives are useful in environments where,
# because of NAT, Sentinel is reachable from outside via a non-local address.
#
# When announce-ip is provided, the Sentinel will claim the specified IP address
# in HELLO messages used to gossip its presence, instead of auto-detecting the
# local address as it usually does.
#
# Similarly when announce-port is provided and is valid and non-zero, Sentinel
# will announce the specified TCP port.
#
# The two options don't need to be used together, if only announce-ip is
# provided, the Sentinel will announce the specified IP and the server port
# as specified by the "port" option. If only announce-port is provided, the
# Sentinel will announce the auto-detected local IP and the specified port.

Believe you need something like this in the template:

<% if @resource.sentinel_announce_ip %>
  sentinel announce-ip <%= @resource.sentinel_announce_ip %>
<% end %>
<% if @resource.sentinel_announce_port %>
  sentinel announce-port <%= @resource.sentinel_announce_port %>
<% end %>
johnbellone commented 7 years ago

I have a PR that will fix this.