From: Stephen Gran <steve@lobefin.net>
Date: Sat, 17 Oct 2009 15:04:44 +0000 (+0100)
Subject: ok, so package relationships aren't working for me yet
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=f021caecf7f1878c55f5e703551874c52d778c86;p=dsa-puppet.git

ok, so package relationships aren't working for me yet

Signed-off-by: Stephen Gran <steve@lobefin.net>
---

diff --git a/modules/apache2/manifests/init.pp b/modules/apache2/manifests/init.pp
index 2a5cd1fc..90c2f824 100644
--- a/modules/apache2/manifests/init.pp
+++ b/modules/apache2/manifests/init.pp
@@ -1,31 +1,29 @@
 class apache2 {
-	define enable_module($ensure=present, $require=apache2) {
+        include munin-node::apache
+
+	package {
+		"apache2": ensure => installed;
+		"logrotate": ensure => installed;
+	}
+
+	define enable_module($ensure=present) {
 	        case $ensure {
 	                present: {
 	                        exec { "/usr/sbin/a2enmod $name":
 	                                unless => "/bin/sh -c '[ -L /etc/apache2/mods-enabled/${name}.load ]'",
 	                                notify => Exec["force-reload-apache2"],
-	                                require => Package[$require],
 	                        }
 	                }
 	                absent: {
 	                        exec { "/usr/sbin/a2dismod $name":
 	                                onlyif => "/bin/sh -c '[ -L /etc/apache2/mods-enabled/${name}.load ]'",
 	                                notify => Exec["force-reload-apache2"],
-	                                require => Package[$require],
 	                        }
 	                }
 	                default: { err ( "Unknown ensure value: '$ensure'" ) }
 	         }
 	}
 
-        include munin-node::apache
-
-	package {
-		apache2: ensure => installed;
-		logrotate: ensure => installed;
-	}
-
         enable_module {
                 "info":;
                 "status":;