]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
Ensure all services have 'hasstatus => true' for Puppet 2.6
authorKen Barber <ken@bob.sh>
Mon, 3 Jun 2013 13:22:53 +0000 (14:22 +0100)
committerKen Barber <ken@bob.sh>
Mon, 3 Jun 2013 13:22:53 +0000 (14:22 +0100)
We were getting reports of idempotency issues with 2.6, due to missing
hasstatus setting.

Signed-off-by: Ken Barber <ken@bob.sh>
manifests/linux/archlinux.pp
manifests/linux/debian.pp
manifests/linux/redhat.pp

index 1ed11602d25affafd9fefa858caa6ab9a2b300d8..ef7991bb73d359cc1d4cc663355521630e51a977 100644 (file)
@@ -5,11 +5,13 @@ class firewall::linux::archlinux (
   service { 'iptables':
     ensure => $ensure,
     enable => $enable,
+    hasstatus => true,
   }
 
   service { 'ip6tables':
     ensure => $ensure,
     enable => $enable,
+    hasstatus => true,
   }
 
   file { '/etc/iptables/iptables.rules':
index 1470893104229e5f79b206a3dc2d556f7f8b96a6..1d60156e4f5d903325c7e93a6ddeb4ae4f6a0657 100644 (file)
@@ -19,8 +19,9 @@ class firewall::linux::debian (
     # This isn't a real service/daemon. The start action loads rules, so just
     # needs to be called on system boot.
     service { 'iptables-persistent':
-      ensure  => undef,
-      enable  => $enable,
+      ensure => undef,
+      enable => $enable,
+      hasstatus => true,
       require => Package['iptables-persistent'],
     }
   }
index c7dcb947d1fda2ff19d27dccbeda7626e23c4279..b31819396f4c8193e5677824125c659a532878ff 100644 (file)
@@ -5,5 +5,6 @@ class firewall::linux::redhat (
   service { 'iptables':
     ensure => $ensure,
     enable => $enable,
+    hasstatus => true,
   }
 }