Skip to content

Commit a35623b

Browse files
jas01jas01
jas01
authored and
jas01
committed
Fix group on FreeBSD
The server::backup class don't use the correct information from mysql::params about backupdirgroup. Because on FreeBSD the group root doesn't exist it should use wheel instead. This difference are correctly configure inside mysql::params but not inside server::backup
1 parent e3acc7d commit a35623b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Diff for: manifests/server/backup.pp

+4-4
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
# @param backupcompress
3232
# Whether or not to compress the backup (when using the mysqldump provider)
3333
# @param backupmethod
34-
# The execution binary for backing up. ex. mysqldump, xtrabackup, mariabackup
34+
# The execution binary for backing up. ex. mysqldump, xtrabackup, mariabackup
3535
# @param backuprotate
3636
# Backup rotation interval in 24 hour periods.
3737
# @param ignore_events
@@ -56,7 +56,7 @@
5656
# @param execpath
5757
# Allows you to set a custom PATH should your MySQL installation be non-standard places. Defaults to `/usr/bin:/usr/sbin:/bin:/sbin`.
5858
# @param provider
59-
# Sets the server backup implementation. Valid values are:
59+
# Sets the server backup implementation. Valid values are:
6060
# @param maxallowedpacket
6161
# Defines the maximum SQL statement size for the backup dump script. The default value is 1MB, as this is the default MySQL Server value.
6262
# @param optional_args
@@ -68,7 +68,7 @@
6868
$backupdir = undef,
6969
$backupdirmode = '0700',
7070
$backupdirowner = 'root',
71-
$backupdirgroup = 'root',
71+
$backupdirgroup = $mysql::params::root_group,
7272
$backupcompress = true,
7373
$backuprotate = 30,
7474
$backupmethod = undef,
@@ -86,7 +86,7 @@
8686
$provider = 'mysqldump',
8787
$maxallowedpacket = '1M',
8888
$optional_args = [],
89-
) {
89+
) inherits mysql::params {
9090

9191
if $prescript and $provider =~ /(mysqldump|mysqlbackup)/ {
9292
warning(translate("The 'prescript' option is not currently implemented for the %{provider} backup provider.",

0 commit comments

Comments
 (0)