Fixes

README.md

@@ -168,7 +168,7 @@ For debian users, the config_include_dir has been defaulted to /etc/mysql/conf.d
     percona::rights {'userbar on dbfoo':
       priv => 'select_priv',
       host => 'localhost',
-      database => '*'
+      database => '*',
       password => 'default',
     }
 

manifests/adminpass.pp

@@ -50,7 +50,7 @@
   exec {"percona-adminpass-${name}":
     onlyif    => [
       'test -f /usr/bin/mysqladmin',
-      "mysqladmin -u${user} -h${host} --no-defaults status",
+      "mysqladmin --no-defaults -u${user} -h${host}  status",
     ],
     path      => ['/usr/bin','/bin',],
     command   => "mysqladmin -h ${host} -u${user} password ${password}",

manifests/init.pp

@@ -46,7 +46,6 @@
 #       percona::rights {'Set rights for replication':
 #         user     => 'repl',
 #         password => 'repl',
-#         database => ['%'],
 #         priv     => ['Repl_slave_priv'],
 #         host     => '$hostname',
 #       }

manifests/rights.pp

@@ -111,8 +111,8 @@
 
   ## Host param
   $_host = $host ? {
-    undef   => $default_host,
-    default => $host,
+    'localhost' => $default_host,
+    default     => $host,
   }
 
   ## Database param

templates/mgmt_cnf.erb

@@ -2,6 +2,10 @@
 ## Served by:        '<%= scope.lookupvar('::servername') %>' 
 ## Module:           '<%= scope.to_hash['module_name'] %>' 
 ## Template source:  'MODULES<%= template_source.gsub(Regexp.new("^#{Puppet::Node::Environment.current[:modulepath].gsub(':','|')}"),"") %>'
+[client]
+  user = <%= user %>
+  password = <%= password %>
+
 [mysql]
   user = <%= user %>
   password = <%= password %>