From 0913456af52fccfa3e8d137cbb0afb743764070c Mon Sep 17 00:00:00 2001 From: sfozz Date: Fri, 24 Aug 2012 12:30:39 +0100 Subject: [PATCH] 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] --- README.markdown | 12 ++++++++++++ 1 file changed, 12 insertions(+) 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 -- 2.45.2