]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
ceph-rgw-loadbalancer: Fix SSL newline issue
authorFlorian Faltermeier <florian.faltermeier@uibk.ac.at>
Wed, 18 Dec 2019 13:31:57 +0000 (14:31 +0100)
committerGuillaume Abrioux <gabrioux@redhat.com>
Mon, 17 Feb 2020 15:05:42 +0000 (16:05 +0100)
The ad7a5da commit introduced a regression when using TLS on haproxy
via the haproxy_frontend_ssl_certificate variable.
This cause the "stats socket" and the "tune.ssl.default-dh-param"
parameters to be on the same line resulting haproxy failing to start.

[ALERT] 351/140240 (21388) : parsing [xxxxx] : 'stats socket' : unknown
keyword 'tune.ssl.default-dh-param'. Registered
[ALERT] 351/140240 (21388) : Fatal errors found in configuration.

Fixes: #4869
Signed-off-by: Florian Faltermeier <florian.faltermeier@uibk.ac.at>
roles/ceph-rgw-loadbalancer/templates/haproxy.cfg.j2

index c7c08b7262556d6ee2fab6d1cb3dbea565a54a1c..0fea9ab5252e0223ffa7afd128c77d99df86af47 100644 (file)
@@ -9,7 +9,7 @@ global
     group       haproxy
     daemon
     stats socket /var/lib/haproxy/stats
-{%- if haproxy_frontend_ssl_certificate %}
+{% if haproxy_frontend_ssl_certificate %}
     tune.ssl.default-dh-param {{ haproxy_ssl_dh_param }}
     ssl-default-bind-ciphers {{ haproxy_ssl_ciphers | join(':') }}
     ssl-default-bind-options {{ haproxy_ssl_options | join(' ') }}