summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Fix #3551 rake spec fails to run integration specsJesse Wolfe2010-02-171-1/+1
| | | | | | A typo prevents the Rakefile from finding tests in spec/integration Signed-off-by: Jesse Wolfe <jes5199@gmail.com>
* Update Red Hat spec file for 0.25.5Todd Zullinger2010-02-171-9/+15
| | | | | | | | With RHEL-6 just around the corner, now is also a good time to adjust the ruby(selinux) conditional to work there. The rundir-perms patch from tarball is applied rather than including it separately in the SRPM. It makes for one less file to track in downstream distros. Other minor changes are pulled in from the Fedora/EPEL spec file.
* Updated CHANGELOG for 0.25.5rc2James Turnbull2010-02-171-0/+27
|
* Fixing #3533 - Removing all transaction cleanupLuke Kanies2010-02-174-57/+0
| | | | | | | | | | It's really slow and has no actual functionality any more, since we just remove the catalogs from memory anyway. This should be a good speed boost for very little effort. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fix for #2910 -- Tidy/matches is too tricky to useMarkus Roberts2010-02-172-30/+59
| | | | | | | | | | | | | The semantic interaction of tidy/matches and tidy/recurse is tricky to get right; it only makes sense to use matches with recursion (a fixed path will either statically match or it won't, no need for a run-time check) but there was nothing to warn users of this fact. To compound matters, the example in the matches parameter doc string even made this mistake. This patch: 1) fixes the doc string; 2) prohibits the use of match without a value of recurse capable of generating files to match, 3) fixes tests that were passing for the wrong reason and adds tests on the prohibition added in (2).
* Bug #3451: Don't leak the terminus class setting from ↵Jesse Wolfe2010-02-172-0/+12
| | | | | | | Puppet::Resource::Catalog's spec This issue causes other specs to fail, because they depend on the default terminus being unchanged.
* Fix to the fix for #3295Markus Roberts2010-02-171-6/+2
| | | | | | The output variable in the inner block wasn't visible in the outer block, and wasn't needed in any case, since the results are returned naturally if you just leave everything alone.
* Write ssh_authorized_keys as userMarkus Roberts2010-02-172-54/+38
| | | | | | | | | | | | This is a targeted fix to the issue of permissions when writing ssh authorized key files by 1) requiring that an existing users be specified on the resource and 2) doing the write as that user. It's based on Michael DeHaan's initial implementation of Luke's idea, but with a number of simplifications (mostly by testing necessary conditions as early as possible so the code isn't cluttered up with a lot of checks). The tests in this version are modified slightly to remove some additional implementation couplings that were added in master.
* Fixing bad testMarkus Roberts2010-02-171-1/+1
| | | | | | | | | | | Actually, just moving the bad around. The test as written depended on the implementation details of Parser#file=; when those changed, it broke the test and this patch fixes the test by updating it's unwarranted assumptions to match the current details. The real solution should probably be to ban all such tests and yank them from the code base without mercy, but that's a task for another day, after we solve world peace and cure the common cold.
* Fix for #3558 -- source file reading speedupMarkus Roberts2010-02-171-7/+1
| | | | | | It's about 10x faster to read the whole file than to read each line and concatenate them (actually, it's O(n) vs. O(n^2), so the exact speedup depends on the file size).
* Fix for #3556 Plussignment value meldingMarkus Roberts2010-02-172-4/+28
| | | | | | | | | | | | | | The plussignment operator was constructing the new parameter value by modifying the param object's value in place (so as to preserve the file and line information for debugging). However, when multiple resources are overridden by the same plussignment this would result in all of the resources sharing the same value (the union of all the prior values and the new value), which is wrong. Instead, we need to give each resource its own copy of the value (e.g., a copy of the param object), which this patch implements. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fixed #3655 - Puppet doesn't find installed packages with portage providerJames Turnbull2010-02-171-2/+2
| | | | Thanks to Benedikt Böhm for the patch
* Fixed #3672 - Error message on duplicate fileserver mounts incorrectJames Turnbull2010-02-171-2/+2
|
* conf/redhat: Add notifempty to logrotate configTodd Zullinger2010-02-171-0/+1
| | | | | We default to logging via syslog so there aren't any puppet logs on many boxes. Causing a weekly restart shouldn't be necessary.
* Fixed stored configuration documentationJames Turnbull2010-02-171-3/+3
|
* Fixes #3653 - Changed default factpath value to better reflect plugins in ↵James Turnbull2010-02-171-1/+1
| | | | modules
* Partial fix to #2837 - changed warning message to debugJames Turnbull2010-02-172-3/+4
|
* Fix #3555 - fix state of purged listsPeter Meier2010-02-171-0/+10
| | | | | As purged is not contained by default within ensureable we need to extend the parameter to properly deal with purged lists.
* Updated Rake tasks to no longer load puppet.rbJames Turnbull2010-02-172-5/+7
|
* Fix #3540 - name methods correctlyPeter Meier2010-02-171-2/+2
| | | | | The "Ensure Parameter" tries to call destroy on the provider and not delete.
* Fixes #3295 - generate() now sets the working directory to the directory ↵Paul Lathrop2010-02-172-2/+45
| | | | | | containing the specified command. Also adds rspec tests for generate().
* Added YARD taskJames Turnbull2010-02-171-0/+11
| | | | | | | | | | | | You need to: gem install yard Then run: rake yard This will generate a "doc" directory containing YARD documentation.
* Update man pages and partial doc fix to #3491James Turnbull2010-02-178-234/+199
|
* Fixed #3532 - Typo in lib/puppet/ssl/host.rbJames Turnbull2010-02-171-1/+1
| | | | Thanks to Jasper Lievisse Adriaanse for the fix.
* Updated version and CHANGELOG to 0.25.5rc1James Turnbull2010-02-173-25/+640
|
* Modify SuSE spec file for 0.25.x and correct shebang lines forAndrew Forgue2010-02-172-12/+30
| | | | puppetd/puppetmasterd
* Fixes #3460 - Makes Puppet FHS compliant by moving /var/puppet to ↵James Turnbull2010-02-1710-22/+14
| | | | /var/lib/puppet
* Fix for #3101 (bug in MRI 1.8.7)Markus Roberts2010-02-171-1/+12
| | | | | | | | | | | Due to a bug in Ruby 1.8.7 net/http will attempt to close a connection that wasn't successfully opened (it's nil), first checking to see if the connection is already close, and thus raising a method missing exception. This bug causes error messages that are confusing / misleading. To get around this, we add a closed? method to nil such that a nil (unopened) connection is always considered closed, allowing the real problem to be reported.
* Fixes #3419. OS X 10.6 Ruby doesn't set supplementary groupsNigel Kersten2010-02-171-2/+26
|
* Revert the guts of #2890Markus Roberts2010-02-176-77/+112
| | | | | | | | | | This patch reverts the semantically significant parts of #2890 due to the issues discussed on #3360 (security concerns when used with autosign, inconsistency between REST & XMLRPC semantics) but leaves the semantically neutral changes (code cleanup, added tests) in place. This patch is intended for 0.25.x, but may also be applied as a step in the resolution of #3450 (refactored #2890, add "remove_certs" flag) in Rolwf.
* Fail gracefully on packages that don't have the HOMEPAGE variable set (e.g. ↵Gábor Vészi2010-02-171-1/+1
| | | | dev-lang/php).
* Fixed #3443 - Typo in mount typeJames Turnbull2010-02-171-2/+3
|
* Fixes #3135 - darwin doesn't support 'mount -o remount'James Turnbull2010-02-171-1/+2
| | | | Thanks for Eric Sorenson for the patch.
* Adding :catalog_terminus settingLuke Kanies2010-02-177-18/+50
| | | | | | | | | | This will rarely be used, but it enables even more architectural flexibility, such as precompiling catalogs and storing them in memcached or equivalent. With this setup, a single host can probably serve all catalogs and you would then just have as many compiling hosts as needed. Signed-off-by: Luke Kanies <luke@reductivelabs.com>
* fixing obsolete comment in puppetdLuke Kanies2010-02-171-3/+3
| | | | Signed-off-by: Luke Kanies <luke@reductivelabs.com>
* Adding support for only using cached catalogsLuke Kanies2010-02-173-2/+26
| | | | | | | | | This basically allows a sysadmin to control when a client will compile a new catalog - with this option enabled, the client will use the cached catalog as long as it has one, only recompiling when run with the option disabled. Signed-off-by: Luke Kanies <luke@reductivelabs.com>
* Refactoring Configurer to enable the next featureLuke Kanies2010-02-171-27/+30
| | | | Signed-off-by: Luke Kanies <luke@reductivelabs.com>
* Fixed changelog Rake taskJames Turnbull2010-02-171-1/+1
|
* Fix #3155 - prevent error when using two matching regex in cascadeBrice Figureau2010-02-178-20/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following manifest: case $var { /match/: { if $var =~ /matchagain/ { } } } is failing because the "=~" operators when matching sets an ephemeral variable in the scope. But the case regex also did it, and since they both belong to the same scope, and Puppet variables are immutables, the scope raises an error. This patch fixes this issue by adding to the current scope a stack of ephemeral symbol tables. Each new match operator or case/selector with regex adds a new scope. When we get out of the case/if/selector structure the scope is reset to the ephemeral level we were when entering it. This way the following manifest produces the correct output: case $var { /match(rematch)/: { notice("1. \$0 = $0, \$1 = $1") if $var =~ /matchagain/ { notice("2. \$0 = $0, \$1 = $1") } notice("3. \$0 = $0, \$1 = $1") } } notice("4. \$0 = $0") And the output is: 1. $0 = match, $1 = rematch 2. $0 = matchagain, $1 = rematch 3. $0 = match, $1 = rematch 4. $0 = Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixed puppetlast typoJames Turnbull2010-02-171-1/+1
| | | | Patch thanks for Micah Anderson
* Fixed README contents to reflect Puppet Labs and docs and wiki changesJames Turnbull2010-04-222-7/+14
|
* Fixed link typoJames Turnbull2010-04-221-1/+1
|
* Fixed #3384 - Updated broken linkJames Turnbull2010-04-221-3/+3
|
* Making a Puppet::Module test more resilientLuke Kanies2010-04-132-3/+4
| | | | | | It would fail if a directory unexpectedly existed. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing yumrepo type to use 'flush'Luke Kanies2010-04-121-25/+1
| | | | | | | | It was previously relying on monkey-patching the last 'Change' instance in 'evaluate', but we removed 'evaluate', so this wasn't working any more. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Only trying to retrieve passwords on OS X when rootLuke Kanies2010-04-121-2/+2
| | | | | | | We were otherwise failing whenever we tried to read a user. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Removing obsolete testsLuke Kanies2010-04-121-64/+1
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing a warning in the aix package providerLuke Kanies2010-04-122-66/+1
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Removing tests for code that was removed in the REST refactorLuke Kanies2010-04-123-340/+0
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing and porting Transaction Report testsLuke Kanies2010-04-125-107/+117
| | | | | | | There are still a few unported tests, but it's at least better now. Signed-off-by: Luke Kanies <luke@puppetlabs.com>