Add support for dynamic backupmethods/mariabackup

Author: Daniel Quackenbush

manifests/backup/mysqlbackup.pp

@@ -13,6 +13,7 @@
   $backupdirgroup     = $mysql::params::root_group,
   $backupcompress     = true,
   $backuprotate       = 30,
+  $backupmethod       = '',
   $ignore_events      = true,
   $delete_before_dump = false,
   $backupdatabases    = [],

manifests/backup/mysqldump.pp

@@ -12,6 +12,7 @@
   $backupdirgroup     = $mysql::params::root_group,
   $backupcompress     = true,
   $backuprotate       = 30,
+  $backupmethod       = 'mysqldump',
   $ignore_events      = true,
   $delete_before_dump = false,
   $backupdatabases    = [],

manifests/backup/xtrabackup.pp

@@ -1,5 +1,5 @@
 # @summary
-#   "Provider" for Percona XtraBackup
+#   "Provider" for Percona XtraBackup/MariaBackup
 # @api private
 #
 class mysql::backup::xtrabackup (
@@ -8,7 +8,7 @@
   $backuppassword          = undef,
   $backupdir               = '',
   $maxallowedpacket        = '1M',
-  $backupmethod            = 'mysqldump',
+  $backupmethod            = 'xtrabackup',
   $backupdirmode           = '0700',
   $backupdirowner          = 'root',
   $backupdirgroup          = $mysql::params::root_group,
@@ -26,7 +26,7 @@
   $postscript              = false,
   $execpath                = '/usr/bin:/usr/sbin:/bin:/sbin',
   $optional_args           = [],
-  $additional_cron_args    = ''
+  $additional_cron_args    = '--backup'
 ) inherits mysql::params {
 
   ensure_packages($xtrabackup_package_name)
@@ -49,7 +49,7 @@
 
   cron { 'xtrabackup-weekly':
     ensure  => $ensure,
-    command => "/usr/local/sbin/xtrabackup.sh ${backupdir} ${additional_cron_args}",
+    command => "/usr/local/sbin/xtrabackup.sh --target-dir=${backupdir} ${additional_cron_args}",
     user    => 'root',
     hour    => $time[0],
     minute  => $time[1],
@@ -59,7 +59,7 @@
 
   cron { 'xtrabackup-daily':
     ensure  => $ensure,
-    command => "/usr/local/sbin/xtrabackup.sh --incremental ${backupdir} ${additional_cron_args}",
+    command => "/usr/local/sbin/xtrabackup.sh --incremental-basedir=${backupdir} --target-dir=${backupdir}/`date +%F_%H-%M-%S` ${additional_cron_args}",
     user    => 'root',
     hour    => $time[0],
     minute  => $time[1],

manifests/server/backup.pp

@@ -64,6 +64,7 @@
   $backupdirgroup     = 'root',
   $backupcompress     = true,
   $backuprotate       = 30,
+  $backupmethod       = undef,
   $ignore_events      = true,
   $delete_before_dump = false,
   $backupdatabases    = [],
@@ -95,6 +96,7 @@
       'backupdirgroup'     => $backupdirgroup,
       'backupcompress'     => $backupcompress,
       'backuprotate'       => $backuprotate,
+      'backupmethod'       => $backupmethod,
       'ignore_events'      => $ignore_events,
       'delete_before_dump' => $delete_before_dump,
       'backupdatabases'    => $backupdatabases,

spec/classes/mysql_server_backup_spec.rb

@@ -363,7 +363,7 @@ class { 'mysql::server': }
 
         it 'contains the wrapper script' do
           is_expected.to contain_file('xtrabackup.sh').with_content(
-            %r{^innobackupex\s+.*?"\$@"},
+            %r{(\n*^xtrabackup\s+.*\$@)},
           )
         end
 
@@ -398,6 +398,19 @@ class { 'mysql::server': }
             )
           end
         end
+        context 'with mariabackup' do
+          let(:params) do
+            default_params.merge(provider: 'xtrabackup',
+                                 backupmethod: 'mariabackup'
+                                )
+          end
+
+          it 'contain the mariabackup executor' do
+            is_expected.to contain_file('xtrabackup.sh').with_content(
+              %r{(\n*^mariabackup\s+.*\$@)},
+            )
+          end
+        end
       end
     end
   end

templates/mysqlbackup.sh.erb

@@ -91,18 +91,18 @@ cleanup
 <% if @file_per_database -%>
 mysql --defaults-extra-file=$TMPFILE -s -r -N -e 'SHOW DATABASES' | while read dbname
 do
-  mysqldump --defaults-extra-file=$TMPFILE --opt --flush-logs --single-transaction \
+  <%= @backupmethod -%> --defaults-extra-file=$TMPFILE --opt --flush-logs --single-transaction \
     ${ADDITIONAL_OPTIONS} \
     ${dbname} <% if @backupcompress %>| bzcat -zc <% end %>> ${DIR}/${PREFIX}${dbname}_`date +%Y%m%d-%H%M%S`.sql<% if @backupcompress %>.bz2<% end  %>
 done
 <% else -%>
-mysqldump --defaults-extra-file=$TMPFILE --opt --flush-logs --single-transaction \
+<%= @backupmethod -%> --defaults-extra-file=$TMPFILE --opt --flush-logs --single-transaction \
  ${ADDITIONAL_OPTIONS} \
  --all-databases <% if @backupcompress %>| bzcat -zc <% end %>> ${DIR}/${PREFIX}`date +%Y%m%d-%H%M%S`.sql<% if @backupcompress %>.bz2<% end  %>
 <% end -%>
 <% else -%>
 <% @backupdatabases.each do |db| -%>
-mysqldump --defaults-extra-file=$TMPFILE --opt --flush-logs --single-transaction \
+<%= @backupmethod -%> --defaults-extra-file=$TMPFILE --opt --flush-logs --single-transaction \
     ${ADDITIONAL_OPTIONS} \
  <%= db %><% if @backupcompress %>| bzcat -zc <% end %>> ${DIR}/${PREFIX}<%= db %>_`date +%Y%m%d-%H%M%S`.sql<% if @backupcompress %>.bz2<% end  %>
 <% end -%>

templates/xtrabackup.sh.erb

@@ -28,7 +28,7 @@
   <%- end -%>
 <%- end -%>
 
-innobackupex <%= _innobackupex_args %> "$@"
+<%= @backupmethod -%> <%= _innobackupex_args %> $@
 
 <% if @postscript -%>
   <%- [@postscript].flatten.compact.each do |script| %>