From 9158a09a09454a317e00176804a0be889ce70830 Mon Sep 17 00:00:00 2001
From: Juan Antonio Osorio Robles <jaosorior@redhat.com>
Date: Fri, 23 Sep 2016 17:26:45 +0300
Subject: Make mysql bind-address configurable

It used to be hardcoded that the bind-address was always coming from
the $::hostname fact. This is wrong, as it disregards where we have
configured the mysql address. This commit actually makes it
configurable, so we'll be able to set it via hieradata.

On the other hand, we use the hiera key that we already set
'mysql_bind_host' as a default; if, for some reason, that's
unavailable then we fall back to $::hostname.

Related-Bug: #1627060
Change-Id: I316acfd514aac63b84890e20283c4ca611ccde8b
---
 manifests/profile/base/database/mysql.pp      | 10 +++++++---
 manifests/profile/pacemaker/database/mysql.pp |  9 +++++++--
 2 files changed, 14 insertions(+), 5 deletions(-)

(limited to 'manifests')

diff --git a/manifests/profile/base/database/mysql.pp b/manifests/profile/base/database/mysql.pp
index 49c9df3..31a90ec 100644
--- a/manifests/profile/base/database/mysql.pp
+++ b/manifests/profile/base/database/mysql.pp
@@ -18,6 +18,10 @@
 #
 # === Parameters
 #
+# [*bind_address*]
+#   (Optional) The address that the local mysql instance should bind to.
+#   Defaults to $::hostname
+#
 # [*manage_resources*]
 #   (Optional) Whether or not manage root user, root my.cnf, and service.
 #   Defaults to true
@@ -37,11 +41,11 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::database::mysql (
+  $bind_address            = $::hostname,
   $manage_resources        = true,
   $mysql_server_options    = {},
   $remove_default_accounts = true,
   $step                    = hiera('step'),
-
 ) {
 
   validate_hash($mysql_server_options)
@@ -59,14 +63,14 @@ class tripleo::profile::base::database::mysql (
     } else {
       $mysql_config_file = '/etc/my.cnf.d/server.cnf'
     }
-    # TODO Galara
+    # TODO Galera
     # FIXME: due to https://bugzilla.redhat.com/show_bug.cgi?id=1298671 we
     # set bind-address to a hostname instead of an ip address; to move Mysql
     # from internal_api on another network we'll have to customize both
     # MysqlNetwork and ControllerHostnameResolveNetwork in ServiceNetMap
     $mysql_server_default = {
       'mysqld' => {
-        'bind-address'     => $::hostname,
+        'bind-address'     => $bind_address,
         'max_connections'  => hiera('mysql_max_connections'),
         'open_files_limit' => '-1',
       }
diff --git a/manifests/profile/pacemaker/database/mysql.pp b/manifests/profile/pacemaker/database/mysql.pp
index 4c54309..c688240 100644
--- a/manifests/profile/pacemaker/database/mysql.pp
+++ b/manifests/profile/pacemaker/database/mysql.pp
@@ -18,13 +18,18 @@
 #
 # === Parameters
 #
+# [*bind_address*]
+#   (Optional) The address that the local mysql instance should bind to.
+#   Defaults to $::hostname
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::database::mysql (
-  $step = hiera('step'),
+  $bind_address = $::hostname,
+  $step         = hiera('step'),
 ) {
   if $::hostname == downcase(hiera('bootstrap_nodeid')) {
     $pacemaker_master = true
@@ -52,7 +57,7 @@ class tripleo::profile::pacemaker::database::mysql (
       'innodb_locks_unsafe_for_binlog'=> '1',
       'query_cache_size'              => '0',
       'query_cache_type'              => '0',
-      'bind-address'                  => $::hostname,
+      'bind-address'                  => $bind_address,
       'max_connections'               => hiera('mysql_max_connections'),
       'open_files_limit'              => '-1',
       'wsrep_on'                      => 'ON',
-- 
cgit 1.2.3-korg