From 017327245107ce0a1fed8c17b4295cdff212a825 Mon Sep 17 00:00:00 2001 From: Sally Young Date: Tue, 11 Jun 2024 12:35:43 +0100 Subject: [PATCH 1/2] Update firewall.pp to replace action with jump https://forge.puppet.com/modules/puppetlabs/firewall/readme#migration-path-to-v700 The action attribute within the firewall type has been removed as it was merely a restricted version of the jump attribute, both of them managing the same function, this being reasoned as a way to enforce the use of generic parameters. From this point the parameters formerly unique to action should now be passed to jump. --- manifests/firewall.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifests/firewall.pp b/manifests/firewall.pp index d5d116ba4..decfe08f1 100644 --- a/manifests/firewall.pp +++ b/manifests/firewall.pp @@ -5,7 +5,7 @@ assert_private() firewall { '500 allow Jenkins inbound traffic': - action => 'accept', + jump => 'accept', state => 'NEW', dport => [jenkins_port()], proto => 'tcp', From b555054b0573d34e0ab3314b7fa547d6c8013fba Mon Sep 17 00:00:00 2001 From: Sally Young Date: Tue, 11 Jun 2024 12:38:09 +0100 Subject: [PATCH 2/2] Fix indentation --- manifests/firewall.pp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/manifests/firewall.pp b/manifests/firewall.pp index decfe08f1..8e88f57c1 100644 --- a/manifests/firewall.pp +++ b/manifests/firewall.pp @@ -5,9 +5,9 @@ assert_private() firewall { '500 allow Jenkins inbound traffic': - jump => 'accept', - state => 'NEW', - dport => [jenkins_port()], - proto => 'tcp', + jump => 'accept', + state => 'NEW', + dport => [jenkins_port()], + proto => 'tcp', } }