X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=modules%2Fferm%2Fmanifests%2Finit.pp;h=781a4a16b12055c81ff4e4ff4c4bb148a1c47bcf;hb=29cbe59430d1e7d7d5575579b48513c56227e2f7;hp=f33c8ba8dc83f147a5b3d04a34f60db49a56240f;hpb=30be0f0db19c688a30cc568c80be00a6724bd1e2;p=mirror%2Fdsa-puppet.git

diff --git a/modules/ferm/manifests/init.pp b/modules/ferm/manifests/init.pp
index f33c8ba8d..781a4a16b 100644
--- a/modules/ferm/manifests/init.pp
+++ b/modules/ferm/manifests/init.pp
@@ -7,10 +7,6 @@
 #   include ferm
 #
 class ferm {
-	# realize (i.e. enable) all @ferm::rule virtual resources
-	Ferm::Rule <| |>
-	Ferm::Conf <| |>
-
 	File { mode => '0400' }
 
 	package { 'ferm':
@@ -28,6 +24,12 @@ class ferm {
 		hasstatus   => false,
 		status      => '/bin/true',
 	}
+	exec {
+		"ferm reload":
+			command     => "service ferm reload",
+			refreshonly => true,
+	}
+
 
 	$munin_ips = getfromhash($site::nodeinfo, 'misc', 'v4addrs')
 		.map |$addr| { "ip_${addr}" }
@@ -40,7 +42,7 @@ class ferm {
 
 	file { '/etc/ferm':
 		ensure  => directory,
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 		require => Package['ferm'],
 		mode    => '0755'
 	}
@@ -63,38 +65,38 @@ class ferm {
 	file { '/etc/default/ferm':
 		source  => 'puppet:///modules/ferm/ferm.default',
 		require => Package['ferm'],
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 		mode    => '0444',
 	}
 	file { '/etc/ferm/ferm.conf':
 		content => template('ferm/ferm.conf.erb'),
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 	}
 	file { '/etc/ferm/conf.d/00-init.conf':
 		content => template('ferm/00-init.conf.erb'),
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 	}
 	file { '/etc/ferm/conf.d/me.conf':
 		content => template('ferm/me.conf.erb'),
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 	}
 	file { '/etc/ferm/conf.d/defs.conf':
 		content => template('ferm/defs.conf.erb'),
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 	}
 
 	file { '/etc/ferm/conf.d/50-munin-interfaces.conf':
 		content => template('ferm/conf.d-munin-interfaces.conf.erb'),
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 	}
-	@ferm::rule { 'dsa-munin-interfaces-in':
+	ferm::rule { 'dsa-munin-interfaces-in':
 		prio        => '001',
 		description => 'munin accounting',
 		chain       => 'INPUT',
 		domain      => '(ip ip6)',
 		rule        => 'daddr ($MUNIN_IPS) NOP'
 	}
-	@ferm::rule { 'dsa-munin-interfaces-out':
+	ferm::rule { 'dsa-munin-interfaces-out':
 		prio        => '001',
 		description => 'munin accounting',
 		chain       => 'OUTPUT',
@@ -104,7 +106,7 @@ class ferm {
 
 	file { '/etc/ferm/dsa.d/010-base.conf':
 		content => template('ferm/dsa.d-010-base.conf.erb'),
-		notify  => Service['ferm'],
+		notify  => Exec['ferm reload'],
 	}
 
 	augeas { 'logrotate_ulogd2':