]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
(#15702) firewall duplicated rules are not removed
authorAnastasis Andronidis <anastasis90@yahoo.gr>
Thu, 26 Jul 2012 12:24:21 +0000 (14:24 +0200)
committerKen Barber <ken@bob.sh>
Mon, 15 Apr 2013 00:07:58 +0000 (01:07 +0100)
This fix makes sure that every rule has a different name so same rules
get deleted.

lib/puppet/provider/firewall/iptables.rb

index edc7a53bbcdfe569c03d23f3f4f94932a5484c60..03da45486614478441fcea4405ad2c08c28ae350 100644 (file)
@@ -232,7 +232,7 @@ Puppet::Type.type(:firewall).provide :iptables, :parent => Puppet::Provider::Fir
     # Puppet-firewall requires that all rules have comments (resource names) and will fail if
     # a rule in iptables does not have a comment. We get around this by appending a high level
     if ! hash[:name]
-      hash[:name] = "9999 #{Digest::MD5.hexdigest(line)}"
+      hash[:name] = "999#{counter} #{Digest::MD5.hexdigest(line)}"
     end
 
     # Iptables defaults to log_level '4', so it is omitted from the output of iptables-save.