Permalink
Browse files

Merge pull request #20 from endemics/master

add information about concat_basedir and the need for pluginsync = true at least once
  • Loading branch information...
2 parents e9c97b8 + 0bbbe32 commit 52dbf4a61a0a47b82b5213def9fe19b7af47a498 @ripienaar ripienaar committed May 20, 2012
Showing with 17 additions and 7 deletions.
  1. +6 −0 README.markdown
  2. +8 −4 manifests/init.pp
  3. +3 −3 manifests/setup.pp
View
6 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:
-------------
View
12 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
View
6 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

0 comments on commit 52dbf4a

Please sign in to comment.