Kafka bind address puppet #27

Closed
wants to merge 3 commits into
from
Jump to file or symbol
Failed to load files and symbols.
+6 −1
Split
@@ -24,7 +24,8 @@
class server(
$broker_id = "0",
$port = "9092",
- $zookeeper_connection_string = "localhost:2181"
+ $zookeeper_connection_string = "localhost:2181",
+ $bind_addr = ""
) {
package { 'kafka':
@@ -25,7 +25,11 @@ broker.id=<%= @broker_id %>
port=<%= @port %>
# Hostname the broker will bind to. If not set, the server will bind to all interfaces
+<% if !@bind_addr.nil? && !@bind_addr.empty? %>
+host.name=<%= @bind_addr %>
@johnsca

johnsca Jul 22, 2016

Owner

host.name is deprecated in favor of listeners in the latest release of Kakfa (0.10.0). This is fine for now, as Bigtop is using 0.8.1, but it's something to keep in mind for the future.

+<% else %>
#host.name=localhost
+<% end %>
# Hostname the broker will advertise to producers and consumers. If not set, it uses the
# value for "host.name" if configured. Otherwise, it will use the value returned from