Remove double tcp_listen_options entries for rabbit
authorMichele Baldessari <michele@acksyn.org>
Wed, 19 Oct 2016 13:02:29 +0000 (15:02 +0200)
committerMichele Baldessari <michele@acksyn.org>
Fri, 21 Oct 2016 05:46:12 +0000 (07:46 +0200)
commitc6ddaafe549e171f4daeab5a627f77712fafa32d
tree0630e1027d7bc4e308eb9eab85d78b46433c9059
parent81aa47d3143bf58cffdb0bd9fbacfd0e05e195e4
Remove double tcp_listen_options entries for rabbit

After a brand new deployment we have the following in rabbitmq.config:
...
  {rabbit, [
    {tcp_listen_options,
         [binary,
         {packet,        raw},
         {reuseaddr,     true},
         {backlog,       128},
         {nodelay,       true},
         {exit_on_close, false}]
    },
    {tcp_listen_options, [binary, {packet, raw}, {reuseaddr, true},
{backlog, 128}, {nodelay, true}, {exit_on_close, false}, {keepalive,
true}]},
...

Let's remove these duplicate entries and make sure that we use the
parameters for the puppet module to set the following values
explicitely (it's the only parameter where we do not use the default
setting from the puppet module):
keepalive = true -> rabbitmq::tcp_keepalive: true

All the other options that we set are the default in the puppet module:
{packet, raw}
{reuseaddr, true}
{backlog, 128}{nodelay, true}
{exit_on_close, false}

Depends-On: I608477d5714a5081b3b4ab3b9fc2932bdd598301
Change-Id: I35921652bd84d1d6be0727051294983d4a0dde10
puppet/services/rabbitmq.yaml