Age | Commit message (Collapse) | Author |
|
|
|
please consider pulling
|
|
|
|
|
|
|
|
|
|
Fixes various puppet-lint warnings.
|
|
* ./manifests/init.pp - WARNING: ensure found on line but it's not the first attribute on line 222
* ./manifests/setup.pp - WARNING: selector inside resource block on line 34
* ./manifests/fragment.pp - WARNING: case statement without a default case on line 28
|
|
Allow using a custom name and provide a path to the file that needs to be created using concat.
|
|
|
|
|
|
|
|
Add puppet-lint and rspec-puppet gems to make Travis CI work again
|
|
|
|
Remove trailing whitespace to make puppet-lint happy
|
|
puppet-lint does not like trailing whitespaces
|
|
Include concat::setup from concat, so users don't need to
|
|
|
|
Remove spurious 'e' character.
|
|
|
|
Fix module name to make the PMT happier
|
|
|
|
Improvements
|
|
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
|
|
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
|
|
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
|
|
Feature/add basic tests
|
|
|
|
Successfully tested inside rvm:
- ruby 1.8.7-p302;
- puppet (2.7.13)
- facter (1.6.9)
- bundler (1.1.3)
- diff-lcs (1.1.3)
- metaclass (0.0.1)
- mocha (0.11.4)
- puppet-lint (0.1.13)
- rake (0.9.2.2)
- rspec (2.8.0)
- rspec-core (2.8.0)
- rspec-expectations (2.8.0)
- rspec-mocks (2.8.0)
- rspec-puppet (0.1.3)
- rubygems-bundler (1.0.2)
- rvm (1.11.3.3)
|
|
|
|
Fail with helpful advice if $::concat_basedir is not yet set
|
|
Building on the new docs in https://github.com/ripienaar/puppet-concat/pull/20
|
|
add information about concat_basedir and the need for pluginsync = true at least once
|
|
least once
|
|
|
|
A patch to make puppet-concat better comply with community style guidelines.
|
|
Comply better with puppet community style guidelines.
Change-Id: Ie1782d266539b1a8f9890b2ade77280011aa22aa
|
|
Remove the gnu parameter from concat
|
|
Previously, the concatfragments.sh script was would default to using
GNU-specific flags for find, sort, and xargs. This necessitated
explicit passing of a "gnu = false" parameter to the concat define in
order to successfully run the script without GNU-specific flags when
working with systems that do not by default include GNU versions of the
utilities (solaris, for example).
This commit modifies the concatfragments script such that GNU versions
of the utilities are not needed at all. It does this while preserving
the original use case for the GNU flags, which was to allow special
characters (like spaces) in the filenames, which it accomplished by
using GNU flags to separate fields using null characters instead of
newlines.
In order to preserve backwards-compatibility with existing puppet
installations that make use of the "gnu = false" parameter, the
parameter list for the concat define has not been changed. Rather, the
gnu parameter is now deprecated and ignored.
|
|
* Zap the old copy of /usr/local/bin/concatfragments.sh.
* Fix comments to point at new location of script.
* Delete trailing whitespace.
|
|
Unset silly Nexenta environment variable.
|
|
Not sure how this is set in our puppet environment, but it is.
http://nexenta.org/projects/site/wiki/Personalities
|
|
Fix use of unqualified variable names
|
|
Fully qualify $::id, $::puppetversion, and $::concat_basedir.
|
|
use client's vardir - make module usable for unprivileged users
|
|
|
|
Fix for /bin/sh on Solaris.
|
|
-a file True if file exists. (Not available
in sh.)
|
|
updated to better support dash
|
|
To be able to use the module as an unprivileged user we need to
adjust certain things:
* only enforce the run user and group if we are root and can
actually change the user
* set owner/group of our files and directories to our own user/group
* place the concat script in a location we can write -> use the
concat dir for that.
|