Minor fixes to appease puppet-lint

Signed-off-by: Wolf Noble <wolf@wolfspyre.com>

diff --git a/manifests/linux.pp b/manifests/linux.pp
index 13ad762bd3ed2e9cb4aa99a224be817caf53cab6..184d9e995b77308b252138916b5c7691e3b9747e 100644 (file)
--- a/manifests/linux.pp
+++ b/manifests/linux.pp
@@ -1,3 +1,5 @@
+#
+#This class includes the proper sub-class for the distro being run
 class firewall::linux (
   $ensure = running
 ) {

diff --git a/manifests/linux/archlinux.pp b/manifests/linux/archlinux.pp
index ef7991bb73d359cc1d4cc663355521630e51a977..317cdb5044a21d6e684924c4ce6246c2c1ee02b4 100644 (file)
--- a/manifests/linux/archlinux.pp
+++ b/manifests/linux/archlinux.pp
@@ -1,16 +1,18 @@
+#
+#This class manages iptables on archlinux
 class firewall::linux::archlinux (
   $ensure = 'running',
   $enable = true
 ) {
   service { 'iptables':
-    ensure => $ensure,
-    enable => $enable,
+    ensure    => $ensure,
+    enable    => $enable,
     hasstatus => true,
   }
 
   service { 'ip6tables':
-    ensure => $ensure,
-    enable => $enable,
+    ensure    => $ensure,
+    enable    => $enable,
     hasstatus => true,
   }
 

diff --git a/manifests/linux/debian.pp b/manifests/linux/debian.pp
index 1d60156e4f5d903325c7e93a6ddeb4ae4f6a0657..bbf666abb2f33780c78bbf11f6d0adffe49eb1ac 100644 (file)
--- a/manifests/linux/debian.pp
+++ b/manifests/linux/debian.pp
@@ -1,3 +1,5 @@
+#
+#This class manages iptables on debian
 class firewall::linux::debian (
   $ensure = running,
   $enable = true
@@ -6,23 +8,23 @@ class firewall::linux::debian (
     ensure => present,
   }
 
-  if($operatingsystemrelease =~ /^6\./ and $enable == true) {
+  if($::operatingsystemrelease =~ /^6\./ and $enable == true) {
     # This fixes a bug in the iptables-persistent LSB headers in 6.x, without it
     # we lose idempotency
     exec { 'iptables-persistent-enable':
       logoutput => on_failure,
-      command => '/usr/sbin/update-rc.d iptables-persistent enable',
-      unless => '/usr/bin/test -f /etc/rcS.d/S*iptables-persistent',
-      require => Package['iptables-persistent'],
+      command   => '/usr/sbin/update-rc.d iptables-persistent enable',
+      unless    => '/usr/bin/test -f /etc/rcS.d/S*iptables-persistent',
+      require   => Package['iptables-persistent'],
     }
   } else {
     # 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'],
+      require   => Package['iptables-persistent'],
     }
   }
 }

diff --git a/manifests/linux/redhat.pp b/manifests/linux/redhat.pp
index b31819396f4c8193e5677824125c659a532878ff..bb917575b8760f99568161e23b9079ccbdacbc38 100644 (file)
--- a/manifests/linux/redhat.pp
+++ b/manifests/linux/redhat.pp
@@ -1,10 +1,12 @@
+#
+#This class manages iptables on redhat
 class firewall::linux::redhat (
   $ensure = running,
   $enable = true
 ) {
   service { 'iptables':
-    ensure => $ensure,
-    enable => $enable,
+    ensure    => $ensure,
+    enable    => $enable,
     hasstatus => true,
   }
 }