Merge pull request #700 from mvisonneau/master

Bugfix on Xtrabackup crons
xbezdick · Apr 22, 2015 · a4b7595 · a4b7595
2 parents 873dea3 + 2b9e777
commit a4b7595
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/manifests/backup/xtrabackup.pp b/manifests/backup/xtrabackup.pp
@@ -19,27 +19,23 @@
   $execpath           = '/usr/bin:/usr/sbin:/bin:/sbin',
 ) {
 
-  mysql_user { "${backupuser}@localhost":
-    ensure        => $ensure,
-    password_hash => mysql_password($backuppassword),
-    require       => Class['mysql::server::root_password'],
-  }
-
   package{ 'percona-xtrabackup':
     ensure  => $ensure,
   }
+
   cron { 'xtrabackup-weekly':
     ensure  => $ensure,
-    command => 'innobackupex $backupdir',
+    command => "innobackupex ${backupdir}",
     user    => 'root',
     hour    => $time[0],
     minute  => $time[1],
     weekday => 0,
     require => Package['percona-xtrabackup'],
   }
+
   cron { 'xtrabackup-daily':
     ensure  => $ensure,
-    command => 'innobackupex --incremental $backupdir',
+    command => "innobackupex --incremental ${backupdir}",
     user    => 'root',
     hour    => $time[0],
     minute  => $time[1],
@@ -54,5 +50,4 @@
     owner  => $backupdirowner,
     group  => $backupdirgroup,
   }
-
 }