From c56631a320fac180d4bc296e5fe60a07634e43fb Mon Sep 17 00:00:00 2001
From: Peter Palfrader <peter@palfrader.org>
Date: Mon, 23 Sep 2019 08:03:27 +0200
Subject: [PATCH] No longer configure apache just because it is installed. 
 Instead, fail if it is installed but not pulled in by puppet

---
 manifests/site.pp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/manifests/site.pp b/manifests/site.pp
index 29ac94ff4..e939261b4 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -41,10 +41,6 @@ node default {
 		include acpi
 	}
 
-	if $::apache2 {
-		include apache2
-	}
-
 	if $::hostname in [buxtehude,milanollo,lw01,lw02,lw03,lw04,lw09,lw10,senfter,gretchaninov,sibelius] {
 		include nfs_server
 	}
@@ -68,4 +64,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.')
+		}
+	}
 }
-- 
2.20.1