x/packstack
Code Issues Proposed changes

Fix several deprecation warnings

Browse Source 
database_user -> mysql_user
mysql_module removal
sql_connection -> database_connection

Needs: https://github.com/redhat-openstack/openstack-puppet-modules/pull/132
Change-Id: Ia7e22b0d809e40e64dc664f281f7274a4102751c
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
This commit is contained in:
Lukas Bezdicka
2014-10-27 15:32:11 +01:00
committed by Gael Chamoulaud
parent 219cf98b4f
commit c4c790a6ae
14 changed files with 70 additions and 82 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packstack/puppet/templates/cinder_qpid.pp
View File

@@ -8,8 +8,7 @@ class {'cinder':
qpid_protocol => hiera('CONFIG_AMQP_PROTOCOL'),
qpid_username => hiera('CONFIG_AMQP_AUTH_USER'),
qpid_password => hiera('CONFIG_AMQP_AUTH_PASSWORD'),
sql_connection => "mysql://cinder:${cinder_qpid_cfg_cinder_db_pw}@${cinder_qpid_cfg_mariadb_host}/cinder",
database_connection => "mysql://cinder:${cinder_qpid_cfg_cinder_db_pw}@${cinder_qpid_cfg_mariadb_host}/cinder",
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
mysql_module => '2.2',
}

3
packstack/puppet/templates/cinder_rabbitmq.pp
View File

@@ -6,8 +6,7 @@ class {'cinder':
rabbit_port => hiera('CONFIG_AMQP_CLIENTS_PORT'),
rabbit_userid => hiera('CONFIG_AMQP_AUTH_USER'),
rabbit_password => hiera('CONFIG_AMQP_AUTH_PASSWORD'),
sql_connection => "mysql://cinder:${cinder_rab_cfg_cinder_db_pw}@${cinder_rab_cfg_mariadb_host}/cinder",
database_connection => "mysql://cinder:${cinder_rab_cfg_cinder_db_pw}@${cinder_rab_cfg_mariadb_host}/cinder",
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
mysql_module => '2.2',
}

6
packstack/puppet/templates/glance.pp
View File

@@ -7,10 +7,9 @@ class { 'glance::api':
keystone_user => 'glance',
keystone_password => hiera('CONFIG_GLANCE_KS_PW'),
pipeline => 'keystone',
sql_connection => "mysql://glance:${glance_ks_pw}@${glance_mariadb_host}/glance",
database_connection => "mysql://glance:${glance_ks_pw}@${glance_mariadb_host}/glance",
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
mysql_module => '2.2',
}
class { 'glance::registry':
@@ -18,9 +17,8 @@ class { 'glance::registry':
keystone_tenant => 'services',
keystone_user => 'glance',
keystone_password => hiera('CONFIG_GLANCE_KS_PW'),
sql_connection => "mysql://glance:${glance_ks_pw}@${glance_mariadb_host}/glance",
database_connection => "mysql://glance:${glance_ks_pw}@${glance_mariadb_host}/glance",
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
mysql_module => '2.2',
}

2
packstack/puppet/templates/heat_qpid.pp
View File

@@ -15,5 +15,5 @@ class { 'heat':
qpid_protocol => hiera('CONFIG_AMQP_PROTOCOL'),
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
sql_connection => "mysql://heat:${heat_qpid_cfg_heat_db_pw}@${heat_qpid_cfg_mariadb_host}/heat",
database_connection => "mysql://heat:${heat_qpid_cfg_heat_db_pw}@${heat_qpid_cfg_mariadb_host}/heat",
}

2
packstack/puppet/templates/heat_rabbitmq.pp
View File

@@ -13,5 +13,5 @@ class { 'heat':
rabbit_password => hiera('CONFIG_AMQP_AUTH_PASSWORD'),
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
sql_connection => "mysql://heat:${heat_rabbitmq_cfg_heat_db_pw}@${heat_rabbitmq_cfg_mariadb_host}/heat",
database_connection => "mysql://heat:${heat_rabbitmq_cfg_heat_db_pw}@${heat_rabbitmq_cfg_mariadb_host}/heat",
}

3
packstack/puppet/templates/keystone.pp
View File

@@ -5,11 +5,10 @@ $keystone_cfg_mariadb_host = hiera('CONFIG_MARIADB_HOST')
class { 'keystone':
admin_token => hiera('CONFIG_KEYSTONE_ADMIN_TOKEN'),
sql_connection => "mysql://keystone_admin:${keystone_cfg_ks_db_pw}@${keystone_cfg_mariadb_host}/keystone",
database_connection => "mysql://keystone_admin:${keystone_cfg_ks_db_pw}@${keystone_cfg_mariadb_host}/keystone",
token_format => hiera('CONFIG_KEYSTONE_TOKEN_FORMAT'),
verbose => true,
debug => hiera('CONFIG_DEBUG_MODE'),
mysql_module => '2.2',
service_name => $keystone_service_name,
enable_ssl => $keystone_use_ssl,
}

1
packstack/puppet/templates/mariadb_cinder_install.pp
View File

@@ -3,5 +3,4 @@ class { 'cinder::db::mysql':
host => '%%',
allowed_hosts => '%%',
charset => 'utf8',
mysql_module => '2.2',
}

1
packstack/puppet/templates/mariadb_glance_install.pp
View File

@@ -3,5 +3,4 @@ class { 'glance::db::mysql':
host => '%%',
allowed_hosts => '%%',
charset => 'utf8',
mysql_module => '2.2',
}

1
packstack/puppet/templates/mariadb_heat_install.pp
View File

@@ -3,5 +3,4 @@ class { 'heat::db::mysql':
host => '%%',
allowed_hosts => '%%',
charset => 'utf8',
mysql_module => '2.2',
}

6
packstack/puppet/templates/mariadb_install.pp
View File

@@ -21,19 +21,19 @@ class { 'mysql::server':
# deleting database users for security
# this is done in mysql::server::account_security but has problems
# when there is no fqdn, so we're defining a slightly different one here
database_user { [ 'root@127.0.0.1', 'root@::1', '@localhost', '@%%' ]:
mysql_user { [ 'root@127.0.0.1', 'root@::1', '@localhost', '@%%' ]:
ensure => 'absent',
require => Class['mysql::server'],
}
if ($::fqdn != '' and $::fqdn != 'localhost') {
database_user { [ "root@${::fqdn}", "@${::fqdn}"]:
mysql_user { [ "root@${::fqdn}", "@${::fqdn}"]:
ensure => 'absent',
require => Class['mysql::server'],
}
}
if ($::fqdn != $::hostname and $::hostname != 'localhost') {
database_user { ["root@${::hostname}", "@${::hostname}"]:
mysql_user { ["root@${::hostname}", "@${::hostname}"]:
ensure => 'absent',
require => Class['mysql::server'],
}

1
packstack/puppet/templates/mariadb_keystone_install.pp
View File

@@ -3,5 +3,4 @@ class { 'keystone::db::mysql':
password => hiera('CONFIG_KEYSTONE_DB_PW'),
allowed_hosts => '%%',
charset => 'utf8',
mysql_module => '2.2',
}

1
packstack/puppet/templates/mariadb_neutron_install.pp
View File

@@ -4,5 +4,4 @@ class { 'neutron::db::mysql':
allowed_hosts => '%%',
dbname => hiera('CONFIG_NEUTRON_L2_DBNAME'),
charset => 'utf8',
mysql_module => '2.2',
}

1
packstack/puppet/templates/mariadb_nova_install.pp
View File

@@ -3,5 +3,4 @@ class { 'nova::db::mysql':
host => '%%',
allowed_hosts => '%%',
charset => 'utf8',
mysql_module => '2.2',
}

3
packstack/puppet/templates/neutron_api.pp
View File

@@ -1,10 +1,9 @@
class { 'neutron::server':
sql_connection => $neutron_sql_connection,
database_connection => $neutron_sql_connection,
connection => $neutron_sql_connection,
auth_password => $neutron_user_password,
auth_host => hiera('CONFIG_CONTROLLER_HOST'),
enabled => true,
mysql_module => '2.2',
}
exec { 'neutron-db-manage upgrade':