From: Daniel Black <grooverdan@users.sourceforge.net>
Date: Sun, 18 Dec 2011 09:16:19 +0000 (+1100)
Subject: (#12897) Require jump=>LOG for log params
X-Git-Tag: 0.1.0~41^2
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=267f5e7f27ff8b0cfaca8cd6ff6a30cc0fef5609;p=puppet-modules%2Fpuppetlabs-firewall.git

(#12897) Require jump=>LOG for log params

This validates that when log_prefix or log_level is specified the jump
should be 'LOG'
---

diff --git a/lib/puppet/type/firewall.rb b/lib/puppet/type/firewall.rb
index f8cc358..4843895 100644
--- a/lib/puppet/type/firewall.rb
+++ b/lib/puppet/type/firewall.rb
@@ -570,6 +570,12 @@ Puppet::Type.newtype(:firewall) do
       end
     end
 
+    if value(:log_prefix) || value(:log_level)
+      unless value(:jump).to_s == "LOG"
+        self.fail "Parameter log_prefix and log_level require jump => LOG"
+      end
+    end
+
     if value(:burst) && ! value(:limit)
       self.fail "burst makes no sense without limit"
     end