From 64951b5c124b90e97404a7ad5c1439a768afbb2a Mon Sep 17 00:00:00 2001 From: Michele Baldessari Date: Tue, 7 Jan 2020 17:36:37 +0100 Subject: [PATCH] Allow mysql haproxy stanza do be customized We introduce a new tripleo::haproxy parameter called mysql_custom_listen_options which can be used to customize mysql listen options in haproxy's config. Tested as follows: parameter_defaults: ExtraConfig: tripleo::haproxy::haproxy_default_timeout: - 'http-request 10s' - 'queue 2m' - 'connect 10s' - 'client 5m' - 'server 5m' - 'check 10s' tripleo::haproxy::cinder::options: bind-process: 2 tripleo::haproxy::glance_api::options: bind-process: 2 tripleo::haproxy::mysql_custom_listen_options: bind-process: 2 Observed that haproxy's mysql stanza had: listen mysql bind 172.17.1.91:3306 transparent bind-process 2 option tcpka option httpchk Before this patch this was not the case and the tripleo::haproxy::::options just got ignored. Closes-Bug: #1858775 Change-Id: Ia0ecca5b54c1ca51898035e641665150dbdac9f7 (cherry picked from commit c42c5aa67828c8277d676e609bc828eb1dd1ba37) --- manifests/haproxy.pp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/manifests/haproxy.pp b/manifests/haproxy.pp index 345d62bc5..a99e46102 100644 --- a/manifests/haproxy.pp +++ b/manifests/haproxy.pp @@ -306,6 +306,10 @@ # and if mysql cluster checking is disabled, the member options will be: "union($haproxy_member_options, ['backup'])" # Defaults to undef # +# [*mysql_custom_listen_options*] +# Hash to pass to the mysql haproxy listen stanza to be deepmerged with the other options +# Defaults to {} +# # [*rabbitmq*] # (optional) Enable or not RabbitMQ binding # Defaults to false @@ -652,6 +656,7 @@ class tripleo::haproxy ( $mysql_clustercheck = false, $mysql_max_conn = undef, $mysql_member_options = undef, + $mysql_custom_listen_options = {}, $openshift_master = hiera('openshift_master_enabled', false), $rabbitmq = false, $etcd = hiera('etcd_enabled', false), @@ -1368,7 +1373,7 @@ class tripleo::haproxy ( } haproxy::listen { 'mysql': bind => $mysql_bind_opts, - options => $mysql_listen_options, + options => deep_merge($mysql_listen_options, $mysql_custom_listen_options), collect_exported => false, } haproxy::balancermember { 'mysql-backup':