refactor everything into its own file

6 files changed, 100 insertions, 8 deletions

diff --git a/manifests/puppetmaster/base.pp b/manifests/puppetmaster/base.pp
new file mode 100644
index 0000000..9d32fc4
--- /dev/null
+++ b/manifests/puppetmaster/base.pp
@@ -0,0 +1,37 @@
+class puppet::puppetmaster::base inherits puppet::base {
+
+    File[puppet_config]{
+        source => [ "puppet://$server/files/puppet/master/puppet.conf",
+                    "puppet://$server/puppet/master/puppet.conf" ],
+        notify => [Service[puppet],Service[puppetmaster] ],
+    }
+
+    $real_puppet_fileserverconfig = $puppet_fileserverconfig ? {
+        '' => "/etc/puppet/fileserver.conf",
+        default => $puppet_fileserverconfig,
+    }
+
+    file { "$real_puppet_fileserverconfig":
+        source => [ "puppet://$server/files/puppet/master/${fqdn}/fileserver.conf",
+                    "puppet://$server/files/puppet/master/fileserver.conf",
+                    "puppet://$server/puppet/master/fileserver.conf" ],
+        notify => [Service[puppet],Service[puppetmaster] ],
+        owner => root, group => 0, mode => 600;
+    }
+
+    if $puppetmaster_storeconfigs {
+        include puppet::puppetmaster::storeconfigs
+    }
+
+    # restart the master from time to time to avoid memory problems
+    file{'/etc/cron.d/puppetmaster.cron':
+        source => [ "puppet://$server/puppet/cron.d/puppetmaster.${operatingsystem}",
+                    "puppet://$server/puppet/cron.d/puppetmaster" ],
+        owner => root, group => 0, mode => 0644;
+    }
+
+    file{'/etc/cron.daily/puppet_reports_cleanup.sh':
+        content => "#!/bin/bash\nfind /var/log/puppet/reports/ -maxdepth 2 -type f -ctime +30 -exec rm {} \\;\n",
+        owner => root, group => 0, mode => 0700;
+    }
+}
diff --git a/manifests/puppetmaster/cluster.pp b/manifests/puppetmaster/cluster.pp
index cfa3fc0..2242ba7 100644
--- a/manifests/puppetmaster/cluster.pp
+++ b/manifests/puppetmaster/cluster.pp
@@ -3,11 +3,3 @@
 class puppet::puppetmaster::cluster inherits puppet::puppetmaster {
     include puppet::puppetmaster::cluster::base
 }
-
-class puppet::puppetmaster::cluster::base inherits puppet::puppetmaster::base {
-    include mongrel, nginx
-
-    File[puppet_config] {
-        require +> [ Package[mongrel], Package[nginx], File[nginx_config] ],
-    }
-}
diff --git a/manifests/puppetmaster/cluster/base.pp b/manifests/puppetmaster/cluster/base.pp
new file mode 100644
index 0000000..759b0d2
--- /dev/null
+++ b/manifests/puppetmaster/cluster/base.pp
@@ -0,0 +1,8 @@
+class puppet::puppetmaster::cluster::base inherits puppet::puppetmaster::base {
+    include mongrel, nginx
+
+    File[puppet_config] {
+        require +> [ Package[mongrel], Package[nginx], File[nginx_config] ],
+    }
+}
+
diff --git a/manifests/puppetmaster/hasdb.pp b/manifests/puppetmaster/hasdb.pp
new file mode 100644
index 0000000..82bbe47
--- /dev/null
+++ b/manifests/puppetmaster/hasdb.pp
@@ -0,0 +1,19 @@
+define puppet::puppetmaster::hasdb(
+    $dbtype = 'mysql',
+    $dbname = 'puppet',
+    $dbhost = 'localhost',
+    # this is needed due to the collection of the databases
+    $dbhostfqdn = "${fqdn}",
+    $dbuser = 'puppet',
+    $dbpwd = $puppet_storeconfig_password,
+    $dbconnectinghost = 'locahost'
+){
+
+    case $puppet_storeconfig_password {
+        '': { fail("No \$puppet_storeconfig_password is set, please set it in your manifests or site.pp to add a password") }
+    }
+
+    case $dbtype {
+        'mysql': {  puppet::puppetmaster::hasdb::mysql{$name: dbname => $dbname, dbhost => $dbhost, dbuser => $dbuser, dbpwd => $dbpwd, } }
+    }
+}
diff --git a/manifests/puppetmaster/hasdb/mysql.pp b/manifests/puppetmaster/hasdb/mysql.pp
new file mode 100644
index 0000000..51fd776
--- /dev/null
+++ b/manifests/puppetmaster/hasdb/mysql.pp
@@ -0,0 +1,31 @@
+# don't use this define use the general interface
+define puppet::puppetmaster::hasdb::mysql(
+    $dbname = 'puppet',
+    $dbhost = 'localhost',
+    $dbhostfqdn = "${fqdn}",
+    $dbuser = 'puppet',
+    $dbpwd,
+    $dbconnectinghost = 'localhost'
+){
+    @@mysql_database{$dbname:
+        tag => "mysql_${dbhostfqdn}",
+    }
+
+    @@mysql_user{"${dbuser}@${dbconnectinghost}":
+        password_hash => mysql_password("$dbpwd"),
+        require => Mysql_database[$dbname],
+        tag => "mysql_${dbhostfqdn}",
+    }
+
+
+    @@mysql_grant{"${dbuser}@${dbconnectinghost}/${dbname}":
+        privileges => all,
+        require => Mysql_user["${dbuser}@${dbconnectinghost}"],
+        tag => "mysql_${dbhostfqdn}",
+    }
+
+    munin::plugin::deploy{'puppetresources':
+        source => "puppet/munin/puppetresources.mysql",
+        config => "env.mysqlopts --user=$dbuser --password=$dbpwd -h $dbhost\nenv.puppetdb $dbname",
+    }
+}
diff --git a/manifests/puppetmaster/storeconfigs.pp b/manifests/puppetmaster/storeconfigs.pp
new file mode 100644
index 0000000..32aaa7f
--- /dev/null
+++ b/manifests/puppetmaster/storeconfigs.pp
@@ -0,0 +1,5 @@
+# This class sets up the necessary ActiveRecord bits
+# so storeconfigs works.
+class puppet::puppetmaster::storeconfigs {
+    include rails
+}