]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
(DOC-3056) Remove mention of rules ordering
authorclairecadman <claire.cadman@puppet.com>
Wed, 30 Jan 2019 14:40:43 +0000 (14:40 +0000)
committerGitHub <noreply@github.com>
Wed, 30 Jan 2019 14:40:43 +0000 (14:40 +0000)
Remove any reference that suggests the ordering number used for rules isn't restricted to the section the rule is in. This is misleading and can lead to mis-configured firewall rules.

See https://tickets.puppetlabs.com/browse/DOC-3056 for more information.

README.markdown

index be53353811214d9182528643f422e40e31e1cc84..dc83bc0de51d91b3dcea9b14b4baffb5caeed7f8 100644 (file)
@@ -62,12 +62,6 @@ The code in this section does the following:
 * The 'require' parameter in `firewall {}` ensures `my_fw::pre` is run before any other rules.
 * In the `my_fw::post` class declaration, the 'before' parameter ensures `my_fw::post` is run after any other rules.
 
-Therefore, the run order is:
-
-* The rules in `my_fw::pre`
-* Your rules (defined in code)
-* The rules in `my_fw::post`
-
 The rules in the `pre` and `post` classes are fairly general. These two classes ensure that you retain connectivity and that you drop unmatched packets appropriately. The rules you define in your manifests are likely to be specific to the applications you run.
 
 1. Add the `pre` class to `my_fw/manifests/pre.pp`, and any default rules to your pre.pp file first — in the order you want them to run.