From: sfozz Date: Fri, 24 Aug 2012 11:30:39 +0000 (+0100) Subject: Add missing class declaration X-Git-Tag: 0.1.0~6^2 X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=0913456af52fccfa3e8d137cbb0afb743764070c;p=puppet-modules%2Fpuppetlabs-firewall.git Add missing class declaration README.markdown was missing details about declaring 'my_fw::pre' and 'my_fw::post' which caused folks following the example to see the following error: Could not find dependency Class[My_w::Pre] for Firewall[BLAH] --- diff --git a/README.markdown b/README.markdown index 663a2cc..3d3daf4 100644 --- a/README.markdown +++ b/README.markdown @@ -112,6 +112,18 @@ So It is recommended that you provide the following in top scope somewhere purge => true } +You also need to declare the 'my_fw::pre' & 'my_fw::post' classes so that +dependencies are satisfied. This can be achieved using an External Node +Classifier or the following:: + + class { 'my_fw::pre': } + class { 'my_fw::post': } + +or: + + include my_fw::pre, my_fw:post + + In this case, it uses classes called 'my_fw::pre' & 'my_fw::post' to define default pre and post rules. These rules are required to run in catalog order to avoid locking yourself out of your own boxes when Puppet runs, as