Refactoring for autoloading

diff --git a/manifests/ca.pp b/manifests/ca.pp
index 8ec6b2ee41a09d510f0834466c53305d8a0da193..61e82b8ad8e49def8f33b04fa05e350c3ad47012 100644 (file)
--- a/manifests/ca.pp
+++ b/manifests/ca.pp
@@ -1,4 +1,4 @@
-class puppetmaster::ca {
+class puppet::ca {
     file {
       '/var/lib/puppetmaster/ssl/ca':
         ensure => directory,

diff --git a/manifests/puppetd.pp b/manifests/daemon.pp
similarity index 99%
rename from manifests/puppetd.pp
rename to manifests/daemon.pp
index 62aad130e05567780ca72a1fe1b8fbc6f4bcc468..27f3651c1f9c14adfe631c771dbb0a6de457727a 100644 (file)
--- a/manifests/puppetd.pp
+++ b/manifests/daemon.pp
@@ -1,5 +1,5 @@
 # handles puppetd service
-class puppetd(
+class puppet::daemon(
   $master = false,
   $main_master = false,
 ) {

diff --git a/manifests/init.pp b/manifests/init.pp
index b434cce4b8259a40a9a20b1c42fb2865f82bdaf1..5dd5dd909ee8f16ef2be6d66cf2c039637676866 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -1,5 +1,48 @@
-# This file imports the files for puppet puppet module.
-import "ca.pp"
-import "puppet.pp"
-import "puppetd.pp"
-import "puppetmasterd.pp"
+# This classes manage a puppet installation, and provide for a puppetd
+# service and a puppetmasterd service, the latter constantly updated by a
+# cron-job which git-pushes from '/var/git/repositories/puppet.git'.
+
+# guarantees puppet user, group and ownerships
+class puppet {
+  user { "puppet":
+    ensure    => present,
+    allowdupe => false,
+  }
+
+  group { "puppet":
+    ensure    => present,
+    allowdupe => false,
+  }
+
+  file { "/etc/puppet":
+    ensure  => directory,
+    owner   => puppet,
+    group   => puppet, 
+    mode    => 0640,
+    require => User["puppet"],
+  }
+
+  file { "/etc/puppet/manifests":
+    ensure  => directory,
+    owner   => puppet,
+    group   => puppet, 
+    recurse => inf,
+    require => User["puppet"],
+  }
+
+  file { "/etc/puppet/modules":
+    ensure  => directory,
+    owner   => puppet,
+    group   => puppet, 
+    recurse => inf,
+    require => User["puppet"],
+  }
+
+  file { "/etc/puppet/templates":
+    ensure  => directory,
+    owner   => puppet,
+    group   => puppet, 
+    recurse => inf,
+    require => User["puppet"],
+  }
+}

diff --git a/manifests/puppetmasterd.pp b/manifests/master.pp
similarity index 99%
rename from manifests/puppetmasterd.pp
rename to manifests/master.pp
index 53e49bc9158a2676f1a4a1412089ed01b0497b33..d3f3e88c80a1b73a7bb6cc022311ef5bd49a770b 100644 (file)
--- a/manifests/puppetmasterd.pp
+++ b/manifests/master.pp
@@ -1,5 +1,5 @@
 # handles puppetmasterd service
-class puppetmasterd(
+class puppet::master(
   $main = false
 ) {
 

diff --git a/manifests/master/mongrel.pp b/manifests/master/mongrel.pp
index 987c33383b6913be78ffb0f4d6ac23e525c4771f..5cdf5510d3cc1aaafa190112229caca66cd95ec2 100644 (file)
--- a/manifests/master/mongrel.pp
+++ b/manifests/master/mongrel.pp
@@ -30,20 +30,3 @@ class puppet::master::mongrel {
   }
 
 }
-
-class puppet::master::mongrel::disabled inherits puppet::master::mongrel {
-  include nginx::puppetmaster::disabled
-
-  Service['puppetmaster'] {
-    enable => false,
-    ensure => stopped,
-  }
-
-  Package['puppetmaster', 'mongrel'] {
-    ensure => absent,
-  }
-
-  File['/etc/cron.d/puppetmaster'] {
-    ensure => absent,
-  }
-}

diff --git a/manifests/master/mongrel/disabled.pp b/manifests/master/mongrel/disabled.pp
new file mode 100644 (file)
index 0000000..84edfd3
--- /dev/null
+++ b/manifests/master/mongrel/disabled.pp
@@ -0,0 +1,16 @@
+class puppet::master::mongrel::disabled inherits puppet::master::mongrel {
+  include nginx::puppetmaster::disabled
+
+  Service['puppetmaster'] {
+    enable => false,
+    ensure => stopped,
+  }
+
+  Package['puppetmaster', 'mongrel'] {
+    ensure => absent,
+  }
+
+  File['/etc/cron.d/puppetmaster'] {
+    ensure => absent,
+  }
+}

diff --git a/manifests/master/passenger.pp b/manifests/master/passenger.pp
index 221a7a2bbf7363694616d10e409368e06ab2faf8..01f1dac39cd02b2937fd6a0ffd89ca651c33e5ab 100644 (file)
--- a/manifests/master/passenger.pp
+++ b/manifests/master/passenger.pp
@@ -4,7 +4,7 @@ class puppet::master::passenger {
   include puppet::master::mongrel::disabled
 
   # then include puppet class
-  include puppetd
+  include puppet::daemon
 
   package { 'puppetmaster-passenger':
     ensure => installed,

diff --git a/manifests/puppet.pp b/manifests/puppet.pp
deleted file mode 100644 (file)
index 5dd5dd9..0000000
--- a/manifests/puppet.pp
+++ /dev/null
@@ -1,48 +0,0 @@
-# This classes manage a puppet installation, and provide for a puppetd
-# service and a puppetmasterd service, the latter constantly updated by a
-# cron-job which git-pushes from '/var/git/repositories/puppet.git'.
-
-# guarantees puppet user, group and ownerships
-class puppet {
-  user { "puppet":
-    ensure    => present,
-    allowdupe => false,
-  }
-
-  group { "puppet":
-    ensure    => present,
-    allowdupe => false,
-  }
-
-  file { "/etc/puppet":
-    ensure  => directory,
-    owner   => puppet,
-    group   => puppet, 
-    mode    => 0640,
-    require => User["puppet"],
-  }
-
-  file { "/etc/puppet/manifests":
-    ensure  => directory,
-    owner   => puppet,
-    group   => puppet, 
-    recurse => inf,
-    require => User["puppet"],
-  }
-
-  file { "/etc/puppet/modules":
-    ensure  => directory,
-    owner   => puppet,
-    group   => puppet, 
-    recurse => inf,
-    require => User["puppet"],
-  }
-
-  file { "/etc/puppet/templates":
-    ensure  => directory,
-    owner   => puppet,
-    group   => puppet, 
-    recurse => inf,
-    require => User["puppet"],
-  }
-}