summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Adding version information to the catalog for compatLuke Kanies2009-09-224-0/+28
| | | | | | | We need to be able to do compatibility testing, and this allows us to do so. Signed-off-by: Luke Kanies <luke@madstop.com>
* Ticket #2626 (unhelpfull error messages)Markus Roberts2009-09-221-2/+4
| | | | | | | | | | Reworks the error message to 1) make it clearer that it's an internal error, not something the user did, 2) rearrange the sentence to make it clearer that "setting" is being used as a noun 3) combined several fields to increase the chance that the identifying information would suffice to lead someone to the actual source of the error. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fixing #2617 - use the cert name as specifiedLuke Kanies2009-09-223-1/+43
| | | | | | | | | | This allows us to search for a cert, and we use the searched-for term as the cert name (for the wrapper, not the actual cert object), rather than the real cert name. This allows us to use symbolic names like 'ca', as we're currently doing. Signed-off-by: Luke Kanies <luke@madstop.com>
* Refactored Puppet packaging and gem creationJames Turnbull2009-09-202-25/+51
|
* FIXES 2616: Remove sync.syncronize and Puppet.infoSteven Jenkins2009-09-201-16/+5
|
* Renamed test branch to testing in rake taskJames Turnbull2009-09-191-4/+4
|
* Fixing #2656 - puppet parseonly tests don't hangLuke Kanies2009-09-191-0/+1
| | | | | | | It was hanging reading stdin because no code had been supplied. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixes #2648. Spurious macauthorization parameter changesNigel Kersten2009-09-191-4/+4
|
* Fix #2652 - Fix SELinux syntax errorJames Turnbull2009-09-181-3/+2
|
* Fixed #2651 - Corrected install permissions on man page directories.James Turnbull2009-09-181-1/+1
|
* Fix #2638 - Allow creating several nodes with matching namesBrice Figureau2009-09-174-3/+24
| | | | | | | | When we are checking if a node exists before creating a new one we were also trying to match with regex node names, finding matches where in fact there is no equality. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Added automatically constructed test branch task and fileJames Turnbull2009-09-172-0/+17
|
* Fix for #2621 (JSON serialization of exec)Markus Roberts2009-09-162-7/+0
| | | | | | | Removed the array wrapping of values for JSON serialization, and the associated test. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fix #2622 - Puppetdoc in single manifest to console is brokenBrice Figureau2009-09-161-3/+5
| | | | | | | | | After the LoadedCode refactoring I forgot to update this (mostly) unused part of puppetdoc which unfortunately is not covered by unit tests. This commit fixes this issue. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #2640 - Daemontools and Runit were not creating the enable symlinkBrice Figureau2009-09-164-16/+31
| | | | | | | | Due to an incorrect tests, those providers weren't enabling themselves when starting, thus failing to create the symlink necessary for them to run. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #2627 - regex node name could lead to invalid tagBrice Figureau2009-09-162-3/+6
| | | | | | | | | We're converting the regex to a straight name to be used as the node class name which later on will be used as tag. It was possible to generate an invalid tag name (containing leading or successive dots). Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Merged fix for #2601Markus Roberts2009-09-162-10/+66
| | | | | | | | | | | This patch rolls up the changeses discussed on the list & the ticket The fqdn_rand now takes any number of additional arguments of any type that has a string representation (typically integers or strings) and concatenats them on to the salt. The tests have been adjusted to reflect this. Signed-off-by: Markus Roberts <Markus@reality.com>
* Updated changelog task and CHANGELOG to version awareJames Turnbull2009-09-152-19116/+4502
|
* Fix for #2637 (Webrick accpting connections on dead sockets)Markus Roberts2009-09-151-1/+8
| | | | | | | | | | There was a race condition between the layers (SSL vs. TCP/IP) that permitted the creation of non-functional connections when webrick managed the connection. This patch moves the responsibility into our code via the provided callbacks and makes sure the socket is valid before accepting the connection. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fixed #2608 - install.rb will not run on ruby 1.9.1 due to ftools being ↵James Turnbull2009-09-151-7/+25
| | | | deprecated
* Merge branch 'tickets/0.25.x/2634' into 0.25.xJames Turnbull2009-09-151-3/+4
|\
| * Fixed #2634 - Added servicegroup_name parameter to serviceescalation typeJames Turnbull2009-09-151-3/+4
| |
* | Fix for #2605 by falling back to alternative solution to #1963Markus Roberts2009-09-151-6/+13
| | | | | | | | | | | | | | | | | | | | Two solutions were proposed and tested for #1963; both worked but one (the read_nonblock solution) was used for performance reasons. This solution does not work on older ruby implementations (1.8.1) because read_nonblock is not available. This patch implements the alternative fix (IO.popen) as a fallback to handles such cases. Signed-off-by: Markus Roberts <Markus@reality.com>
* | Make regsubst() function operate on arrays (feature #2491).Thomas Bellman2009-09-152-30/+124
| | | | | | | | | | | | | | | | | | | | | | Allow the first argument to the regsubst() function be an array, and perform regexp replacement on each element of the array in that case. This patch also adds more error checking to give better error messages to the user when given bad parameters. Signed-off-by: Thomas Bellman <bellman@nsc.liu.se>
* | Fix for #2600 (wrong number of arguments under older mongrel)Markus Roberts2009-09-154-24/+44
|/ | | | | | | | | | | | | | | | | | This was an API compatibility problem with mongrel's HTTPResponse.start() method between Mongrel 1.0.x and 1.1.x (the number of parameters changed). The older version does not provide the option to set the response header message which was used (redundantly with the response body) to return the error message when the HTTP response was signaling an error. In order to suport the older version the call was wrapped with a fallback and the coresponding code in the other rest implementations was adjusted to always send the error message in the response body. Then the rest terminus was adjusted to pull the message from the response body (if it is present) rather than from the header (which is only used as a fallback for dealing with older puppetmasters), and the tests were augmeted to verify this behaviour. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fixs #2620 authconf interpolation, #2570 0-9 in domain namesMarkus Roberts2009-09-142-81/+106
| | | | | | | | | | | | | | Partial refactoring to clean up the case in the ticket (host name containing dots, begining and ending with a digit, was mistaken for an IP address) and a range of related edge cases. Stopped short of a full refactoring (put off to 0.26 as #2623) Added tests for numerous edge cases. This also fixes the issue raised in #2570. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fix for ticket #2618 (stubbing so redhat tests run under debian)Markus Roberts2009-09-091-0/+2
| | | | | | | | This adds stubs so that the service script used by redhat appears to exist while the tests are running, regardless of the operating system being used to test. Signed-off-by: Markus Roberts <Markus@reality.com>
* vim: match regexp language featuresMarc Fournier2009-09-091-1/+4
| | | | Signed-off-by: Marc Fournier <marc.fournier@camptocamp.com>
* Require active_record/version to support ActiveRecord < 2.3Todd Zullinger2009-09-091-0/+1
| | | | | | | | | ActiveRecord < 2.3.0 did not autoload active_record/version which caused puppet to incorrectly believe ActiveRecord was not available. This compliments 1a5c5b3 (Fixing #2508 - removing mention of ActiveRecord 2.3). Signed-off-by: Todd Zullinger <tmz@pobox.com>
* Fixed #2607 - Added Facter dependency for Puppet GemJames Turnbull2009-09-081-0/+1
|
* Updated and created new CHANGELOG formatJames Turnbull2009-09-054-19811/+20315
|
* Adding tests for the #2599 fixLuke Kanies2009-09-051-0/+11
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Ticket #2525 don't fail find_manifest on invalid module namesMarkus Roberts2009-09-052-14/+12
| | | | | | | | | The patch that put validity assertions in for module names broke find_manifest because rather than returning a failure it now rasies an exception. This patch catches the exception and treats it as a negative result. Signed-off-by: Markus Roberts <Markus@reality.com>
* Updated permissions on test filesJames Turnbull2009-09-055-0/+0
|
* Refactoring tests to reduce code size, increase coverage, and makeMarkus Roberts2009-09-052-111/+49
| | | | Luke happy.
* This further normalizes the handling of init-style services (includingMarkus Roberts2009-09-055-58/+298
| | | | | | | | | the redhat "service" wrapper script). Removes special case handling of non-zero exit code in redhat (base already did this) and centralizes scattered @resource[:has_____] checks. Tests that proper versions of each are called and one level of fallbacks. Signed-off-by: Markus Roberts <Markus@reality.com>
* Combined fix for #2525, #2552 -- RedHat service issuesMarkus Roberts2009-09-051-1/+9
| | | | | | | | | This patch normalizes the structure of the RH service routines which should clear up any lingering issues; xxxcmd routines always return an appropriate array, while the coresponding routines (status/restart/ etc.) either call super or take the needed actions. Signed-off-by: Markus Roberts <markus@phage.local>
* Fixed #2589 - Renamed zfs delete to destroy and added testsJames Turnbull2009-09-052-3/+15
|
* Monkey patch to improve yaml compatibility between ruby versionsMarkus Roberts2009-09-052-0/+44
| | | | | | | | | Ruby 1.8.1 can not parse the yanl produced by later versions because it requires explict type tagging of symbols. This patch adds the tagging on to later versions so that mixed version instalations can use yaml. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fixed typo in parser testJames Turnbull2009-09-051-1/+1
|
* Updated Red Hat spec file and RH patches for 0.25.0.James Turnbull2009-09-055-109/+69
| | | | Thanks to Todd Zullinger for the patch and the updates.
* Fixing #2592 - you can escape slashes in regexesLuke Kanies2009-09-042-3/+28
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2590 - modulepath is not cached inappropriatelyLuke Kanies2009-09-032-0/+13
| | | | | | | | | | | | | | | | It was getting cached by Environment instances too early, thus causing some executables to use the default even when it was overridden. We're taking the slightly extreme step of clearing the environment list if any parameter is set, but this is relatively inexpensive and is the only way to always be correct. The reason that the environments cache this value at all is that to build up their module path they have to search through every known module for a lib or plugins directory, which is then done every time a class is sought in the language. Caching it saves a *ton* of file accesses. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed #2593: puppet init script status command not returning proper exit codeJohn A. Barbuto2009-09-041-0/+2
| | | | Signed-off-by: John A. Barbuto <jbarbuto@corp.sourceforge.com>
* Update documentation string to reflect actual intent of ↵Steven Jenkins2009-09-041-1/+1
| | | | Puppet::Node::Facts::Rest
* Fixes #2581. Use new 10.6 global launchd overrides file for service ↵Nigel Kersten2009-09-032-10/+140
| | | | status/enabled
* Fixed Naginator linkJames Turnbull2009-09-031-1/+1
|
* Fixing #2582 - / no longer autorequires /Luke Kanies2009-09-012-3/+21
| | | | | | This was obviously resulting in a dep cycle. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2577 - clarifying and demoting the deprecation noticeLuke Kanies2009-09-014-28/+40
| | | | | | | It's now just notice instead of a warning, and it clarifies that 0.24 clients can't be present. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2574 - autoloading finds plugins in modulesLuke Kanies2009-09-013-7/+40
| | | | | | | | We had some stupid errors that were preventing this from happening; this fixes them and adds an integration test. Signed-off-by: Luke Kanies <luke@madstop.com>