summaryrefslogtreecommitdiffstats
path: root/lib/puppet
Commit message (Collapse)AuthorAgeFilesLines
* Fixing #1631 - adding /sbin and /usr/sbin to PATHLuke Kanies2009-03-241-0/+9
| | | | | | | This is a trivial fix but seems to crop up more often than it should. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed #2004 - ssh_authorized_key fails if no target is definedFrancois Deppierraz2009-03-242-14/+15
| | | | | | | This commit depends on 7f291afdacf59f762c3b78481f5420ec8919e46d (fixing #1629) which was cherry-picked from master. Signed-off-by: Francois Deppierraz <francois@ctrlaltdel.ch>
* Fix Bug #1629Francois Deppierraz2009-03-241-12/+52
| | | | | A refactoring of ssh_authorized_key parsed provider was needed and tests were improved. flush method has been split for clarity.
* Fix #1807 - make Puppet::Util::Package.versioncmp a module functionBrice Figureau2009-03-241-0/+2
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #1829 - Add puppet function versioncmp to compare versionsBrice Figureau2009-03-241-0/+10
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixes incorrect detail variable in OS X version check, re-patches ralsh to ↵Nigel Kersten2009-03-201-3/+3
| | | | work with Facter values and adds error check for missing password hash files.
* Fix #1828 - Scope.number? wasn't strict enough and could produce wrong resultsBrice Figureau2009-03-141-3/+3
| | | | | | | Some invalid numbers were treated as numbers and conversion to Integer was failing returning 0 (for instance 0.24.7). Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixed #2000 - No default specified for checksumJames Turnbull2009-03-091-1/+7
|
* Fixing change printing when list properties are absentLuke Kanies2009-03-071-1/+5
| | | | | | | They were throwing an exception when the 'is' value was 'absent'. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed #2026 - Red Hat ignoring stop methodJames Turnbull2009-03-071-4/+4
|
* Bring in the documentation changes from the master branchBryan Kearney2009-03-041-3/+2
|
* Added a force option to ensure the change is always applied, and call augeas ↵Bryan Kearney2009-03-042-55/+76
| | | | twice to reduce the chance that data is lost
* Backport the fix for #1835Bryan Kearney2009-03-041-4/+22
|
* First cut at the not running if augeas does not change any of the ↵Bryan Kearney2009-03-041-25/+76
| | | | underlieing files
* Bug 1948: Added patch by jab to support the correct ins syntax. Updated the ↵Bryan Kearney2009-03-041-5/+3
| | | | test cases as well
* Fixing #1991 - ldap booleans get converted to booleansLuke Kanies2009-03-051-0/+25
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Refactored a method: extracted about five other methodsLuke Kanies2009-03-051-39/+50
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix #2010 - add protection code for some storeconfig corruptionBrice Figureau2009-03-041-14/+28
| | | | | | | | | | | | | | This patch adds a more robust and self-healing storedconfig in case of logically corrupted database as the one in #2010 (where multiple resources of same references are present in the database for the same host). The problem is that the resources are stored in a hash with the resource ref as the key, so we collapse resource of different id but same reference. The patch fixed this by using a hash by resource id, and maintaining a list of old extraneous resource in the db that are removved after the storeconfig pass. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixing #2013 - prefetching had a mismatch between type and titleLuke Kanies2009-03-041-1/+1
| | | | | | | | | The ParsedFile types seem to be the main one that suffers from this, but the transactions were using the resource titles, not names, so resources were often not getting prefetched correctly. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed report reference pageJames Turnbull2009-03-031-0/+1
|
* Make puppetd --waitforcert option behave as documented:Paul Lathrop2009-02-281-1/+6
| | | | | | | | "You can turn off waiting for certificates by specifying a time of 0." Also add a test to ensure we catch any future regression of this behavior. Signed-off-by: Paul Lathrop <paul@tertiusfamily.net>
* Adding a performance optimization to the FileCollection.Luke Kanies2009-02-281-3/+5
| | | | | | | I saw about a 7x speed increase when adding this simple hash to speed up the file index lookup. Signed-off-by: Luke Kanies <luke@madstop.com>
* Using the FileCollection where appropriate.Luke Kanies2009-02-289-10/+33
| | | | | | | | | | | | | This commit just replaces the :file and :line accessors with the use of the new FileCollection Lookup module. This should mean that we've normalized all file names in a given process, which *might* have drastic RAM improvements. For initial simplicity, I've gone with a single global collection of file names, but it's built so it's easy to use individual file collections instead. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding a FileCollection and a lookup module for it.Luke Kanies2009-02-282-0/+36
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed #1963 - Failing to read /proc/mounts for selinux kills file downloadsJames Turnbull2009-02-281-1/+3
|
* Fixed #2025 - gentoo service provider handle only default init levelJames Turnbull2009-02-271-1/+1
|
* Updated useradd.rb managehome confine to include other RH-like distributionsJames Turnbull2009-02-251-1/+1
|
* Use Puppet.debug instead of own debug flagPeter Meier2009-02-221-7/+5
| | | | | | It's better to use puppet's logging environment than an own. Especially if the default is quite verbose and can't be set by config flag.
* Fixing #558 - File checksums no longer refer to 'nosum'Luke Kanies2009-02-201-34/+0
| | | | | | | It was only used in a method that apparently was not in use any more. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #1871 once and for all - contents are never printedLuke Kanies2009-02-181-8/+11
| | | | | | | | | They were still being printed in noop mode. The fix was to use is_to_s and should_to_s methods, rather than 'change_to_s'. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix #1972 - ActiveRecord fixes resulted in broken testsBrice Figureau2009-02-151-3/+4
| | | | | | | Fix #1930 was not complete, with some typos and misuse of unused code paths in the regular puppetmaster use case. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixing password validation to support symbols.Luke Kanies2009-02-141-1/+1
| | | | | | | My fix for #1920 broke when you set :absent or :present. Signed-off-by: Luke Kanies <luke@madstop.com>
* Only backing up within parsedfile when managing filesLuke Kanies2009-02-142-5/+7
| | | | | | | | | | | I was getting failing tests because I was using non-files for testing and they didn't back up the same, not surprisingly. This moves the 'backup' method to the :flat filetype and then only backs up if the filetype supports it. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing a syntax error in the up2date providerLuke Kanies2009-02-141-1/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Cleaned up variable names to be more sane, clarified error messages and ↵James Turnbull2009-02-141-6/+6
| | | | fixed incorrect use of 'value' variable rather than 'member'.
* Provide dscl -url output support for OS X 10.4 clients using the ↵Nigel Kersten2009-02-141-36/+120
| | | | | | | | | | | | | | | | | directoryservice provider. We refactored the directoryservice provider in puppet 0.24.7 to use the -plist option so that output could be parsed more easily to resolve a bug with values with spaces in the name. We missed that 10.4 does not support this flag, so this patch adds -url output support back in for OS X 10.4 clients only as well as a new -url output parser that copes with spaces. 10.5 clients continue to use the -plist method. Also includes some miscellaneous cleanup of methods and removal of extraneous comments. Signed-off-by: Nigel Kersten <nigelk@google.com> Don't use sw_vers to determine OS X versions, use the Facter value instead. This relies upon another patch to ralsh to ensure that facts are loaded.
* Fix launchd service provider so it is backwards compatible with OS X 10.4 as ↵Nigel Kersten2009-02-141-4/+13
| | | | | | well Clean up fix to launchd service provider to be more concise
* Updated Augeas type codeJames Turnbull2009-02-141-3/+5
|
* Fixed #1831 - Added sprintf functionJames Turnbull2009-02-141-0/+17
|
* Fixed #1830 - Added regsubst functionJames Turnbull2009-02-141-0/+93
|
* Bug 1948: Add logic and testing for the command parsing logicBryan Kearney2009-02-141-5/+6
|
* Updated up2date and service confines to add support for Oracle EL and VMJames Turnbull2009-02-132-3/+5
|
* Fixing #1964 - Facts get loaded from pluginsLuke Kanies2009-02-121-2/+6
| | | | | | | | Applying slightly modified patch. Also added tests. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding a post-processor for Nagios names.Luke Kanies2009-02-131-2/+3
| | | | | | | | | | | | This is a brutal hack until Puppet correctly supports multiple primary keys. It basically just comments out _naginator_name before writing to disk, and uncomments it when reading. This allows Puppet to use it while Nagios ignores it. Yes, a stupid hack, but it appears to work. Signed-off-by: Luke Kanies <luke@madstop.com>
* Revert "Refixing #1420 - _naginator_name is only used for services"Luke Kanies2009-02-131-10/+15
| | | | | | | | This reverts commit efb5cc50c42bc27aec9409e723e3a717ed58c0a8. Conflicts: CHANGELOG
* Fixing #1541 - ParsedFile only backs up files once per transactionLuke Kanies2009-02-132-1/+14
| | | | | | | | This moves responsibility for backups from the filetype to the consumer of the filetype, but only ParsedFile actually uses filetypes. Signed-off-by: Luke Kanies <luke@madstop.com>
* Removing the apparently obsolete netinfo filetype.Luke Kanies2009-02-131-88/+0
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Migrated FileType tests to spec, and fleshed them out a bit.Luke Kanies2009-02-131-3/+3
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Bug #1948: Added patch by jab to support the correct ins syntax. Updated the ↵Bryan Kearney2009-02-131-6/+26
| | | | test cases as well
* Fixing #961 - closing the http connection after every xmlrpc callLuke Kanies2009-02-121-1/+3
| | | | | | | | There were apparently some circumstances that resulted in the connection not being closed; this just closes it every time if it's still open after the rpc call is complete. Signed-off-by: Luke Kanies <luke@madstop.com>