X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=manifests%2Fsite.pp;h=f6bce78ee441aee69b2a5e5f5ef5605db7faa7cf;hb=refs%2Fheads%2Fsalsa;hp=b735738c292bb192a2057acc42478611743f2f1f;hpb=e4818dbab72dc4aaeb120b3dbae082f99f8b83d7;p=mirror%2Fdsa-puppet.git

diff --git a/manifests/site.pp b/manifests/site.pp
index b735738c2..f6bce78ee 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -19,40 +19,20 @@ Service {
 }
 
 node default {
-	include site
-	include munin
-	include syslog_ng
-	include sudo
-	include ssh
-	include debian_org
-	include monit
-	include time
-	include ssl
-	include hardware
-	include nagios::client
-	include resolv
-	include roles
-	include motd
-	include unbound
-	include bacula::client
-	include autofs
-	include dsa_lvm
-	include grub
-	include multipath
-	include popcon
-	include portforwarder
-	include postgres
-	include haveged
-	include huge_mem
-	include tcp_bbr
-	include certregen::client
+	# we really should rename this one
+	include deprecated
+	include base
+
+	# this is magic: it will include whatever classes says we should
+	# include, based on the value of the "classes" array
+	hiera_include('classes')
 
 	include roles::pubsub::client
 	class { 'roles::udldap::client':
 		ensure => absent
 	}
 
-	if getfromhash($site::nodeinfo, 'ganeti') {
+	if getfromhash($deprecated::nodeinfo, 'ganeti') {
 		include ganeti2
 	}
 
@@ -61,23 +41,7 @@ node default {
 		include acpi
 	}
 
-	if $::mta == 'exim4' {
-		if getfromhash($site::nodeinfo, 'heavy_exim') {
-			include exim::mx
-		} else {
-			include exim
-		}
-	} elsif $::mta == 'postfix' {
-		include postfix
-	} else {
-		include exim
-	}
-
-	if $::apache2 {
-		include apache2
-	}
-
-	if $::hostname in [buxtehude,milanollo,lw01,lw02,lw03,lw04,lw09,lw10,senfter,gretchaninov,sibelius] {
+	if $::hostname in [buxtehude,milanollo,lw01,lw02,lw03,lw04,lw09,lw10,gretchaninov,sibelius] {
 		include nfs_server
 	}
 
@@ -89,10 +53,6 @@ node default {
 		include samhain
 	}
 
-	if $::hostname in [geo3,wieck] {
-		include debian_org::radvd
-	}
-
 	if $::spamd {
 		munin::check { 'spamassassin': }
 	}
@@ -100,4 +60,10 @@ node default {
 	if $::hostname == 'pkgmirror-csail' {
 		include varnish_pkgmirror
 	}
+
+	if $::apache2 {
+		if ! defined(Class['apache2']) {
+			fail('We have apache installed but no role pulled it in.')
+		}
+	}
 }