diff --git a/manifests/backup.pp b/manifests/backup.pp index ac63222..b5107ea 100644 --- a/manifests/backup.pp +++ b/manifests/backup.pp @@ -7,13 +7,16 @@ # /usr/bin. # define mysql_backup::backup ( - $minute = '0', - $hour = '0', - $day = '*', - $dest_dir = '/var/backups/mysql_backups', - $rotation = 'daily', - $num_backups = '30', - $defaults_file = '/etc/mysql/debian.cnf' + # The parameters below are grouped in violation of style guide + # to save readable configuration of cron. All other parameters + # are grouped properly. + $day = '*', + $hour = '0', + $minute = '0', + $defaults_file = '/etc/mysql/debian.cnf', + $dest_dir = '/var/backups/mysql_backups', + $num_backups = '30', + $rotation = 'daily', ) { # Wrap in check as there may be mutliple backup defines backing # up to the same dir. diff --git a/manifests/backup_remote.pp b/manifests/backup_remote.pp index 1098039..839b371 100644 --- a/manifests/backup_remote.pp +++ b/manifests/backup_remote.pp @@ -8,14 +8,17 @@ # define mysql_backup::backup_remote ( $database_host, - $database_user, $database_password, - $minute = '0', - $hour = '0', - $day = '*', - $dest_dir = '/var/backups/mysql_backups', - $rotation = 'daily', - $num_backups = '30' + $database_user, + # The parameters below are grouped in violation of style guide + # to save readable configuration of cron. All other parameters + # are grouped properly. + $day = '*', + $hour = '0', + $minute = '0', + $dest_dir = '/var/backups/mysql_backups', + $num_backups = '30', + $rotation = 'daily', ) { # Wrap in check as there may be mutliple backup defines backing # up to the same dir.