opuppet/module-puppetlabs-mysql
5
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Made the 'TRIGGER'privilege of mysqldump backups depend on whether or not we are actually backing up triggers

Browse source
This commit is contained in:
Steven C. Saliman 2015-04-30 12:37:38 -06:00
parent ec14b87a81
commit 49f273a42c
2 changed files with 25 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
manifests/backup/mysqldump.pp
View file

@ -26,6 +26,7 @@ class mysql::backup::mysqldump (
require => Class['mysql::server::root_password'],
}
if $include_triggers {
mysql_grant { "${backupuser}@localhost/*.*":
ensure => $ensure,
user => "${backupuser}@localhost",
@ -33,6 +34,15 @@ class mysql::backup::mysqldump (
privileges => [ 'SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW', 'PROCESS', 'TRIGGER' ],
require => Mysql_user["${backupuser}@localhost"],
}
} else {
mysql_grant { "${backupuser}@localhost/*.*":
ensure => $ensure,
user => "${backupuser}@localhost",
table => '*.*',
privileges => [ 'SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW', 'PROCESS' ],
require => Mysql_user["${backupuser}@localhost"],
}
}
cron { 'mysql-backup':
ensure => $ensure,

9
spec/classes/mysql_server_backup_spec.rb
View file

@ -27,6 +27,15 @@ describe 'mysql::server::backup' do
:privileges => ['SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW', 'PROCESS', 'TRIGGER']
).that_requires('Mysql_user[testuser@localhost]') }
context 'with triggers excluded' do
let(:params) do
{ :include_triggers => false }.merge(default_params)
end
it { is_expected.to contain_mysql_grant('testuser@localhost/*.*').with(
:privileges => ['SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW', 'PROCESS']
).that_requires('Mysql_user[testuser@localhost]') }
end
it { is_expected.to contain_cron('mysql-backup').with(
:command => '/usr/local/sbin/mysqlbackup.sh',
:ensure => 'present'