opendev/system-config
Code Issues Proposed changes

Puppet dashboard mysqld innodb_file_per_table.

Browse Source 
In 232747f we attempted to implement it this way initially, but
subsequent patchsets went off into the weeds instead trying to use
mysql::server::config which, as it turns out, was not introduced
until a newer version of the puppetlabs mysql module than we use.
Rather than go through a potentially disruptive infrastructure-wide
upgrade right this moment, this patch gets the job done in a more
traditional fashion.

Change-Id: I6967388072c7c8b3e170048e23618ea95bccfecc
Reviewed-on: https://review.openstack.org/27511
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Anita Kuno <anita.kuno@enovance.com>
Reviewed-by: Khai Do <zaro0508@gmail.com>
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
This commit is contained in:
Jeremy Stanley 2013-04-25 18:55:47 +00:00 committed by Jenkins
parent 3089973311
commit fec46f5170
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/openstack_project/files/dashboard/mysqld_innodb_fpt.cnf Normal file
View File

@ -0,0 +1,2 @@
[mysqld]
innodb_file_per_table

8
modules/openstack_project/manifests/dashboard.pp
View File

@ -23,8 +23,12 @@ class openstack_project::dashboard(
passenger => true, passenger => true,
} }
mysql::server::config { '50_innodb_file_per_table': file { '/etc/mysql/conf.d/mysqld_innodb_fpt.cnf':
settings => '[mysqld]\ninnodb_file_per_table\n', ensure => present,
source =>
'puppet://modules/openstack_project/dashboard/mysqld_innodb_fpt.cnf',
require => Class['mysql::server'],
notify => Service['mysql'],
} }
} }