Fix gerrit database backups
The gerrit database is now remote, or at least we have a hostname for it, so we should use the backup_remote class. Also, we don't make the dbbackups files anymore, so stop trying to compress them. Change-Id: Iddbaaef9b9fd089f074abeeb988f7cd052c80d25
This commit is contained in:
parent
79cd2bd745
commit
470c3e45fa
@ -23,10 +23,7 @@ class gerrit::cron(
|
||||
}
|
||||
|
||||
cron { 'removedbdumps':
|
||||
ensure => 'absent',
|
||||
user => 'gerrit2',
|
||||
hour => '5',
|
||||
minute => '1',
|
||||
command => 'find /home/gerrit2/dbupdates/ -name "*.sql.gz" -mtime +30 -exec rm -f {} \;',
|
||||
environment => 'PATH=/usr/bin:/bin:/usr/sbin:/sbin',
|
||||
}
|
||||
}
|
||||
|
@ -191,8 +191,11 @@ class openstack_project::gerrit (
|
||||
require => Class[openstack_project::server],
|
||||
}
|
||||
|
||||
mysql_backup::backup { 'gerrit':
|
||||
require => Class['::gerrit'],
|
||||
mysql_backup::backup_remote { 'gerrit':
|
||||
database_host => $mysql_host,
|
||||
database_user => 'gerrit2',
|
||||
database_password => $mysql_password,
|
||||
require => Class['::gerrit'],
|
||||
}
|
||||
|
||||
if ($testmode == false) {
|
||||
|
Loading…
Reference in New Issue
Block a user