]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
(MODULES-2783) Missing ip6tables service name
authorabednarik <alejandro.bednarik@gmail.com>
Wed, 4 Nov 2015 22:02:56 +0000 (19:02 -0300)
committerabednarik <alejandro.bednarik@gmail.com>
Fri, 8 Jan 2016 12:13:53 +0000 (09:13 -0300)
Replaced hardcoded iptables service references with $service_name
variable.

manifests/linux/redhat.pp

index 3144ae7a9754e812a757364aaf17379bcc079f9e..680c04b139dc469e5093001dbd79a21be36f29e0 100644 (file)
@@ -46,7 +46,7 @@ class firewall::linux::redhat (
       exec { '/usr/bin/systemctl daemon-reload':
         require => Package[$package_name],
         before  => Service[$service_name],
-        unless  => '/usr/bin/systemctl is-active iptables'
+        unless  => "/usr/bin/systemctl is-active ${service_name}",
       }
     }
   }
@@ -55,7 +55,7 @@ class firewall::linux::redhat (
     ensure    => $ensure,
     enable    => $enable,
     hasstatus => true,
-    require   => File['/etc/sysconfig/iptables'],
+    require   => File["/etc/sysconfig/${service_name}"],
   }
 
   # Redhat 7 selinux user context for /etc/sysconfig/iptables is set to unconfined_u
@@ -71,7 +71,7 @@ class firewall::linux::redhat (
     default:     { $seluser = undef }
   }
 
-  file { '/etc/sysconfig/iptables':
+  file { "/etc/sysconfig/${service_name}":
     ensure  => present,
     owner   => 'root',
     group   => 'root',