Merge pull request #500 from nerdlich/patch-1

igalic · igalic · commit aa82f83acec0 · 2014-04-22T14:22:54.000+02:00
User needs PROCESS privilege when doing file-per-database backup
diff --git a/manifests/server/backup.pp b/manifests/server/backup.pp
@@ -29,7 +29,7 @@
     ensure     => present,
     user       => "${backupuser}@localhost",
     table      => '*.*',
-    privileges => [ 'SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW' ],
+    privileges => [ 'SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW', 'PROCESS' ],
     require    => Mysql_user["${backupuser}@localhost"],
   }
 
diff --git a/spec/classes/mysql_server_backup_spec.rb b/spec/classes/mysql_server_backup_spec.rb
@@ -19,7 +19,7 @@
         )}
 
         it { should contain_mysql_grant('testuser@localhost/*.*').with(
-            :privileges => ["SELECT", "RELOAD", "LOCK TABLES", "SHOW VIEW"]
+            :privileges => ["SELECT", "RELOAD", "LOCK TABLES", "SHOW VIEW", "PROCESS"]
         )}
 
         it { should contain_cron('mysql-backup').with(

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@`
`29`	`29`	`ensure => present,`
`30`	`30`	`user => "${backupuser}@localhost",`
`31`	`31`	`table => '.',`
`32`		`- privileges => [ 'SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW' ],`
	`32`	`+ privileges => [ 'SELECT', 'RELOAD', 'LOCK TABLES', 'SHOW VIEW', 'PROCESS' ],`
`33`	`33`	`require => Mysql_user["${backupuser}@localhost"],`
`34`	`34`	`}`
`35`	`35`