add information about concat_basedir and the need for pluginsync = true at least...

diff --git a/README.markdown b/README.markdown
index 1945150dc6c128362445fbfdf38f9c9c82d6ffb7..bb9571aa92e78b51579c77c12e9c234ecce20b90 100644 (file)
--- a/README.markdown
+++ b/README.markdown
@@ -81,6 +81,12 @@ Known Issues:
   in a run, the next run will also see it as changed.  This is due
   to how 0.24.8 does the purging of unhandled files, this is improved
   in 0.25.x and we cannot work around it in our code.
+* Since puppet-concat now relies on a fact for the concat directory,
+  you will need to set up pluginsync = true for at least the first run.
+  You have this issue if puppet fails to run on the client and you have
+  a message similar to
+  "err: Failed to apply catalog: Parameter path failed: File
+  paths must be fully qualified, not 'undef' at [...]/concat/manifests/setup.pp:44".
 
 Contributors:
 -------------

diff --git a/manifests/init.pp b/manifests/init.pp
index 48aeec3c25fe3f5575142dce981552ccfe4d3ad5..0b3ed56499540badd221b3e811d09112a0369178 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -31,10 +31,14 @@
 # to control that and thus control the order the final file gets built in.
 #
 # SETUP:
-# The class concat::setup defines a variable $concatdir - you should set this
-# to a directory where you want all the temporary files and fragments to be
-# stored.  Avoid placing this somewhere like /tmp since you should never
-# delete files here, puppet will manage them.
+# The class concat::setup uses the fact concat_basedir to define the variable
+# $concatdir, where all the temporary files and fragments will be
+# durably stored. The fact concat_basedir will be set up on the client to
+# <Puppet[:vardir]>/concat, so you will be able to run different setup/flavours
+# of puppet clients.
+# However, since this requires the file lib/facter/concat_basedir.rb to be
+# deployed on the clients, so you will have to set "pluginsync = true" on
+# both the master and client, at least for the first run.
 #
 # There's some regular expression magic to figure out the puppet version but
 # if you're on an older 0.24 version just set $puppetversion = 24

diff --git a/manifests/setup.pp b/manifests/setup.pp
index ce3f7ea8c4e8f9368161e04ad24602583ec366f7..fb755842e885aa391c73d92bbaa5f357c2aa0b80 100644 (file)
--- a/manifests/setup.pp
+++ b/manifests/setup.pp
@@ -1,8 +1,8 @@
 # Sets up the concat system.
 #
-# $concatdir should point to a place where you wish the fragments to
-# live. This should not be somewhere like /tmp since ideally these files
-# should not be deleted ever, puppet should always manage them
+# $concatdir is where the fragments live and is set on the fact concat_basedir.
+# Since puppet should always manage files in $concatdir and they should
+# not be deleted ever, /tmp is not an option.
 #
 # $puppetversion should be either 24 or 25 to enable a 24 compatible
 # mode, in 24 mode you might see phantom notifies this is a side effect