summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | Move puppetmasterd to Puppet::ApplicationBrice Figureau2009-02-163-166/+474
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Move puppet to the Application Controller paradigmBrice Figureau2009-02-163-189/+449
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Move filebucket to the Application Controller paradigmBrice Figureau2009-02-163-118/+326
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Move puppetca to the Application Controller paradigmBrice Figureau2009-02-163-101/+210
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Introducing the Application ControllerBrice Figureau2009-02-163-0/+725
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fixing broken filetype tests resulting from the loss of Type[]Luke Kanies2009-02-141-11/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing some tests that somehow broke in the merge to masterLuke Kanies2009-02-142-4/+9
| | | | | | | | | | | | | | | | | | | | I expect these failed in master but not 0.24.x because I'm better about stubbing calls to Facter in master. I stubbed these enough that they should continue to work. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Removing a test that was too dependant on order.Luke Kanies2009-02-141-4/+0
| | | | | | | | | | | | | | It wasn't a terribly useful test anyway, since no one depends on that individual configuration. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2009-02-141-2/+2
|\|
| * Minor fix to launchd testsJames Turnbull2009-02-151-2/+2
| |
* | Merge branch '0.24.x'Luke Kanies2009-02-1413-106/+186
|\| | | | | | | | | Conflicts: test/ral/manager/type.rb
| * 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>
| * Fix spec test for launchd service provider to work with new service status ↵Nigel Kersten2009-02-151-1/+9
| | | | | | | | | | | | method and add two new status tests. Now ensures that given the correct input, the correct status for the launchd service is returned.
| * Fixing a test I broke in commit:"897539e857b0da9145f15648b6aa2ef124ec1a19".Luke Kanies2009-02-141-3/+0
| | | | | | | | | | | | I hadn't removed the call to 'prefetch' on the instance in the test. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Removing a no-longer-valid test.Luke Kanies2009-02-141-40/+0
| | | | | | | | | | | | | | The fix for #1884 removed this no-longer-needed feature, so this test is now unnecessary. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing password validation to support symbols.Luke Kanies2009-02-142-5/+9
| | | | | | | | | | | | | | 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-143-8/+19
| | | | | | | | | | | | | | | | | | | | | | 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>
| * Fixing a test broken by a log demotionLuke Kanies2009-02-141-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Cleaned up variable names to be more sane, clarified error messages and ↵James Turnbull2009-02-142-6/+9
| | | | | | | | 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
* | Failure to find node facts is now a failure.Luke Kanies2009-02-142-4/+13
| | | | | | | | | | | | | | | | It was previously just a warning, but the node is essentially non-functional without the facts, so it makes more sense for it to be a warning. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing #1527 - Failing Facter does not hurt PuppetLuke Kanies2009-02-144-2/+30
| | | | | | | | | | | | | | | | At this point, the server's behaviour is a bit undefined if it tries to compile the catalog with no facts locally. The next commits will fix that. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Removing some unused codeLuke Kanies2009-02-141-11/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Using Puppet::Type.new instead of createLuke Kanies2009-02-142-11/+11
| | | | | | | | | | | | | | create() got deprecated and I apparently missed changing this. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Actualling syncing facts and pluginsLuke Kanies2009-02-146-4/+14
| | | | | | | | | | | | | | | | Also fixing the argument order while downloading either of them. I had my Downloader.new calls using the wrong argument order. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fix #1933 - Inconsistent resource evaluation order in subsequent evaluation runsBrice Figureau2009-02-142-10/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | While evaluating the AST, catalog vertices are not always ordered the same way on different run, leading to some tags (which should have been applied in evaluation order) to not be associated with some underlying resources. This changeset change all accesses to resources inside the compiler to always use an ordered (in evaluation order) list of added resources. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Merge branch '0.24.x'Luke Kanies2009-02-1325-344/+856
|\| | | | | | | | | | | | | | | | | | | Conflicts: lib/puppet/indirector/facts/facter.rb lib/puppet/provider/augeas/augeas.rb lib/puppet/util/filetype.rb spec/unit/indirector/facts/facter.rb spec/unit/provider/augeas/augeas.rb test/util/filetype.rb
| * Updated Augeas type codeJames Turnbull2009-02-141-3/+5
| |
| * Removed site from Puppet VIM syntaxJames Turnbull2009-02-141-1/+1
| |
| * Fixed #1831 - Added sprintf functionJames Turnbull2009-02-143-0/+61
| |
| * Fixed #1830 - Added regsubst functionJames Turnbull2009-02-143-0/+183
| |
| * Bug 1948: Add logic and testing for the command parsing logicBryan Kearney2009-02-142-9/+19
| |
| * Updated up2date and service confines to add support for Oracle EL and VMJames Turnbull2009-02-133-3/+7
| |
| * Fixing #1964 - Facts get loaded from pluginsLuke Kanies2009-02-122-4/+26
| | | | | | | | | | | | | | | | Applying slightly modified patch. Also added tests. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Adding tests for Puppet::Indirector::Facts::Facter.loadfactsLuke Kanies2009-02-121-2/+9
| | | | | | | | | | | | | | I just copied the tests from the master branch, changed as necessary. 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-134-7/+50
| | | | | | | | | | | | | | | | 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-132-134/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Migrated FileType tests to spec, and fleshed them out a bit.Luke Kanies2009-02-133-94/+119
| | | | | | | | 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-133-18/+41
| | | | | | | | test cases as well
| * Fixing #961 - closing the http connection after every xmlrpc callLuke Kanies2009-02-123-1/+29
| | | | | | | | | | | | | | | | 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>
| * Refactoring the XMLRPC::Client error-handlingLuke Kanies2009-02-122-60/+254
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I split it all into smaller, manageable chunks, and used methods for each step, instead of having one huge call. Note that I made all of the tests first, then refactored the code, so I'm confident there's no behavior change. I don't know that this is actually a lot cleaner, but it seems that way to me. I'm open to skipping this, but I think it makes the whole thing a lot cleaner. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixed #1959 - Added column protection for environment schema migrationJames Turnbull2009-02-122-2/+8
| |
| * Fixing #1869 - autoloaded files should never leak exceptionsLuke Kanies2009-02-123-2/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | Ruby's exception hierarchy is a bit strange, in that only exceptions that sub RuntimeError are caught by default. This patch explicitly catches the base class, Exception, which means that LoadError, SyntaxError, and any RuntimeErrors will all be caught. This is done for both load() and loadall(); load() uses Kernel.load, but loadall() uses Kernel.require. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2009-02-1132-402/+1308
|\| | | | | | | | | | | Conflicts: CHANGELOG spec/unit/type/file/selinux.rb
| * Fixing #1543 - Nagios parse errors no longer kill PuppetLuke Kanies2009-02-114-65/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The problem wasn't actually transactions, it was how exceptions were raised in Naginator. Well, parse errors actually resulted in an 'exit', rather than an exception, and the exceptions that Naginator was raising were not caught by a normal begin block (SyntaxError, rather than RuntimeError). This commit raises a RuntimeError-derived error rather than exiting. It also adds some context to the error when Puppet catches it. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Moving the transaction specs to the right pathLuke Kanies2009-02-111-1/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>