summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* 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>
* Fixing Property#change_to_s in rare case failuresLuke Kanies2010-04-122-4/+18
| | | | | | | | | If we were removing a field, which really only 'cron' does, then we got an exception. This is fixed, and now tested. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fix test in parser.rb due to API changeJesse Wolfe2010-04-121-1/+1
| | | | Signed-off-by: Jesse Wolfe <jes5199@gmail.com>
* Fixing Hash functionality with non-constant keysLuke Kanies2010-04-122-6/+29
| | | | | | | It was only apparently working with constant keys, not, say, AST strings. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Removing vistigial method in ASTHashLuke Kanies2010-04-121-3/+0
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Porting/removing group test/unit testsLuke Kanies2010-04-123-172/+20
| | | | | | | Most of it was just complicated, and the rest useless. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Porting a simple set of tests from test/unitLuke Kanies2010-04-122-63/+19
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Removing obsolete testLuke Kanies2010-04-121-39/+0
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing Transaction integration testLuke Kanies2010-04-121-1/+1
| | | | | | | This was failing because it was expecting a call to Puppet::Type#evaluate, which is no longer used. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Removing invalid File integration testLuke Kanies2010-04-121-29/+2
| | | | | | | | | | | The point of this test was to confirm that we could take a given file resource and have it work multiple times in memory, but that's not actually possible given our current code. We copy the values over from the source metadata, and suddenly have no method for differentiation between values set by the user and those set by the remote source. Signed-off-by: Luke Kanies <luke@puppetlabs.com>