make trocla an optional dependency

README

@@ -32,11 +32,13 @@ On a node where you wish to have a mysql server installed, you should include
 mysql::server, for example:
 
 node foo {
-  include mysql::server
+  class{'mysql::server':
+    root_password => 'foo',
+  }
 }
 
 This will manage the necessary directories and configuration files, it will
-install the mysql client program and set the root password taken from trocla,
+install the mysql client program and set the root password to 'foo',
 along with setting a /root/.my.cnf for various module operations. It will also
 make sure the mysql service is running, and setup all the databases, users and
 grant tables.
@@ -67,17 +69,19 @@ hiera, before you include mysql::server. This will be used to
 setup a mysql user for munin, with reduced privileges to allow for the various
 munin graphs to be setup and queried. The munin graphs are: mysql_bytes,
 mysql_queries, mysql_slowqueries and mysql_threads. NOTE: The
-munin_mysql_password will be taken from trocla, but it is not necessary on
+munin_mysql_password will be taken from what you passed to the mysql::server
-Debian systems as it will handled with Debian's /etc/mysql/debian.cnf.
+class, but it is not necessary on Debian systems as it will handled with
+Debian's /etc/mysql/debian.cnf.
 
 Nagios
 ------
 
 If you wish nagios to check mysql, you should set the variable "use_nagios" to 
 "true" in hiera along with the "nagios_check_mysql" variable to "true". A 
-password for the nagios mysql user which will be automatically created via trocla
+password for the nagios mysql user will be created for you with reduced privileges
-for you with reduced privileges used only for nagios checks. These should be
+used only for nagios checks. This will be what you passed as nagios_password_hash
-set before you include mysql::server.
+to mysql::server and should be a mysql md5 hash. These should be set before you
+include mysql::server.
 
 Unless you specify otherwise, the default nagios check which will be performed
 is the basic 'check_mysql' nagios plugin which simply tests connectivity to a

manifests/server.pp

@@ -1,8 +1,11 @@
 # manage a mysql server
 class mysql::server (
+  $root_password,
   $manage_shorewall     = false,
   $manage_munin         = false,
+  $munin_password       = 'absent',
   $manage_nagios        = false,
+  $nagios_password_hash = 'absent',
   $backup_cron          = false,
   $optimize_cron        = false,
   $backup_dir           = '/var/backups/mysql',
@@ -17,6 +20,9 @@ class mysql::server (
   }
 
   if $manage_munin and $::mysql_exists == 'true' {
+    if $munin_password == 'absent' {
+      fail("need to set the munin password")
+    }
     case $::operatingsystem {
       debian:  { include mysql::server::munin::debian }
       default: { include mysql::server::munin::default }
@@ -24,6 +30,8 @@ class mysql::server (
   }
 
   if $manage_nagios and $::mysql_exists == 'true' {
+    if $nagios_password_hash == 'absent' {
+      fail("need to set the nagios password hash")
     include mysql::server::nagios
   }
 

manifests/server/munin/default.pp

@@ -1,7 +1,7 @@
 # manage plugins
 class mysql::server::munin::default {
   mysql_user{'munin@localhost':
-    password_hash => trocla("mysql_munin_${::fqdn}",'mysql','length: 32'),
+    password_hash => mysql_password($mysql::server::munin_password),
     require       => Exec['mysql_set_rootpw'],
   }
 
@@ -10,18 +10,17 @@ class mysql::server::munin::default {
     require     => Mysql_user['munin@localhost'],
   }
 
-  $munin_mysql_password = trocla("mysql_munin_${::fqdn}",'plain', 'length: 32')
   munin::plugin {
     [mysql_queries, mysql_slowqueries]:
-      config  => "env.mysqlopts --user=munin --password='${munin_mysql_password}' -h localhost",
+      config  => "env.mysqlopts --user=munin --password='${mysql::server::munin_password}' -h localhost",
       require => Mysql_grant['munin@localhost'];
     [mysql_bytes, mysql_threads]:
-      config  => "env.mysqlopts --user=munin --password=${munin_mysql_password} -h localhost",
+      config  => "env.mysqlopts --user=munin --password=${mysql::server::munin_password} -h localhost",
       require => Mysql_grant['munin@localhost'];
   }
 
   Munin::Plugin::Deploy{
-      config  => "env.mysqlopts --user=munin --password='${munin_mysql_password}' -h localhost",
+      config  => "env.mysqlopts --user=munin --password='${mysql::server::munin_password}' -h localhost",
       require => Mysql_grant['munin@localhost'],
   }
   munin::plugin::deploy{

manifests/server/nagios.pp

@@ -13,7 +13,7 @@ class mysql::server::nagios {
   }
 
   mysql_user{$nagios_mysql_user:
-    password_hash => trocla("mysql_nagios_${::fqdn}",'mysql','length: 32'),
+    password_hash => $mysql::server::nagios_password_hash,
     require       => Package['mysql'],
   }
 

templates/root/my.cnf.erb

@@ -1,4 +1,4 @@
 [client]
 user=root
 host=localhost
-password=<%= Puppet::Parser::Functions.function('trocla');'' %><%= scope.function_trocla("mysql_root_#{scope.lookupvar('::fqdn')}",'plain', 'length' => 32) %>
+password=<%= scope.lookupvar('mysql::server::root_password') %>