summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Fixing puppetmasterd tests when missing rackLuke Kanies2009-05-271-1/+5
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing the Agent so puppetrun actually works server-sideLuke Kanies2009-05-272-2/+10
| | | | | | | We weren't correctly propagating options through to the Configurer instance. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2248 - --no-client correctly leaves off clientLuke Kanies2009-05-272-2/+9
| | | | | | | All of the settings were there, we just weren't doing anything about them. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2243 - puppetrun works againLuke Kanies2009-05-272-1/+16
| | | | | | | | The problem was that some defaults were nil but had to be 'false', because xmlrpc can't serialize 'nil' as an rpc argument. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed #2304 - Added naggen script to directly generate nagios configuration ↵James Turnbull2009-05-281-0/+302
| | | | files from a StoreConfigs Rails database
* Sync conf/redhat/puppet.spec with Fedora/EPELTodd Zullinger2009-05-281-32/+36
| | | | | The install.rb script is now used for installation. The spec file is also updated for the 0.25.0beta1 release.
* Fixed #2280 - Detailed exit codes fixJames Turnbull2009-05-273-2/+38
|
* Fixing #2253 - pluginsync failures propagate correctlyLuke Kanies2009-05-262-8/+33
| | | | | | | | | | | This keeps the destination directory from getting purged if the remote source is invalid. This mostly just removes an optimization that worked fine when we queried the server for every directory, but doesn't work now that we do one big query. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing a transaction test that had some broken plumbingLuke Kanies2009-05-261-2/+11
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Refactoring resource generation slightlyLuke Kanies2009-05-262-5/+5
| | | | | | | | | | | I found some cases where duplicate resources weren't correctly skipped, but I couldn't get the test to really demonstrate them. The code at least is demonstrated to work, anyway. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding a Spec lib directory and moving tmpfile to itLuke Kanies2009-05-265-18/+23
| | | | | | | | | | | | | | We had a common pattern for creating a temporary file during integration tests, and this just makes that common pattern explicit by moving it to a module in the newly-created lib directory in the spec directory. We definitely don't want to go overboard in using libraries in our tests, but sometimes it gets a bit excessive to completely avoid them. Signed-off-by: Luke Kanies <luke@madstop.com>
* Extracting a method from eval_resource in TransactionLuke Kanies2009-05-261-37/+45
| | | | | | Just cleaning up the code a bit before a modification. Signed-off-by: Luke Kanies <luke@madstop.com>
* Not trying to load files that get removed in pluginsyncingLuke Kanies2009-05-262-0/+13
| | | | | | | Previously any changed file got loaded; now we only try to load files that are still present. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix #2300 - Update ssh_authorized_key documentationFrancois Deppierraz2009-05-261-3/+8
|
* Changed version to allow Rake to work. MinorJames Turnbull2009-05-262-1/+7
| | | | edit to Rakefile
* enable maillist on centos, redhat, fedoraPeter Meier2009-05-221-4/+8
| | | | | | | | Fedora and RedHat (so CentOS as well) put the mailman data and binaries in different locations than other systems (looks like the provider have been mainly for debian so far). For the former os we set explicit paths for the different used binaries, so this provider works now as well on those.
* Fixing #2288 - fixing the tests broken by my attr_ttl codeLuke Kanies2009-05-222-0/+10
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix for #2234: test fails with old Rack versionChristian Hofstaedtler2009-05-205-6/+28
|
* Fixing #2273 - file purging works more intuitivelyLuke Kanies2009-05-204-31/+44
| | | | | | | | | | | | | | It now correctly purges files whether we're recursing locally or remotely. *Please* test various scenarios you can think of with this. I've tested: * Local recursion with no remote source * Remote recursion with a source * Recursion with an extra locally managed file Signed-off-by: Luke Kanies <luke@madstop.com>
* Caching whether named autoloaded files are missingLuke Kanies2009-05-204-3/+49
| | | | | | | | This is the big win, because it causes us to just skip the whole loading infrastructure, including skipping looking through the modulepath. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding caching of file metadata to the autoloaderLuke Kanies2009-05-204-3/+242
| | | | | | | | | | | The cache isn't actually used yet - this just adds all of the plumbing. It was found that stat'ing files that didn't exist could take up to 85% of a run, so this is progress toward getting rid of those stats. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding modulepath caching to the AutoloaderLuke Kanies2009-05-203-49/+71
| | | | | | | | | There's more caching to add, but this simplifies the interface to the list of paths and then caches that list so we aren't constantly searching the filesystem. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding caching to the Environment classLuke Kanies2009-05-202-2/+34
| | | | | | | | | | | Caching the module path (because we check which directories exist, and this method can get called often), and the complete list of modules. The cache ttl uses the filetimeout, which defaults to 15 seconds. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding TTL support to attribute cachingLuke Kanies2009-05-202-5/+87
| | | | | | | | | Previously you had to have an Expirer, but now you can declare a TTL for a cached attribute and it will be expired automatically when the cached value is older than the ttl. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixed #2666 - Broken docstring formattingJames Turnbull2009-05-204-9/+19
|
* Deprecating factsync - pluginsync should be used insteadLuke Kanies2009-05-202-0/+11
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Added spec and unit tests to the Rakefile files list and fixed CI rake tasksJames Turnbull2009-05-201-11/+19
|
* Added install.rb to Rakefile package taskJames Turnbull2009-05-201-0/+1
|
* Fixed #2271 - Fix to puppetd documentationJames Turnbull2009-05-181-1/+1
|
* Protecting Stomp client against internal failuresLuke Kanies2009-05-182-1/+10
| | | | | | | | Apparently the stomp client is really unhelpful with failures; this attempts to provide at least a bit more information. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding some usability bits to puppetqdLuke Kanies2009-05-181-3/+21
| | | | | | | There's better logging, and it's a bit more exception-friendly. Signed-off-by: Luke Kanies <luke@madstop.com>
* Refactoring the stomp client and tests a bitLuke Kanies2009-05-182-34/+99
| | | | | | | | | | | | | The main goal of this refactor is to tell the client to be resilient to failures (configured at initialization time), and to send all messages as persistent messages (configured for each message). In the process, the client now parses the queue source URI and handles each argument separately. The tests are more thorough, also. Signed-off-by: Luke Kanies <luke@madstop.com>
* Relying on threads rather than sleeping for puppetqdLuke Kanies2009-05-182-7/+4
| | | | | | | | We previously manually slept, but this uses the queue client to handle keeping the process running, by just joining all running threads. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2250 - Missing templates throw a helpful errorLuke Kanies2009-05-184-25/+38
| | | | | | | | | | | This changes the behaviour of template searching a bit - we previously usually returned a file name, whether the template existed or not. Now we only return a path if it exists. Refactoring a few of the the tests for TemplateWrapper, also. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2273 - CA location is set correctly in puppetcaLuke Kanies2009-05-172-1/+7
| | | | | | It's set to 'only' instead of 'local'. Signed-off-by: Luke Kanies <luke@madstop.com>
* RackXMLRPC: buffer request contents in memory, as a real string.Christian Hofstaedtler2009-05-151-1/+1
| | | | Fixes #2268 "Rack::RewindableInput is not a valid input stream."
* Modules now can find their own pathsLuke Kanies2009-05-1510-123/+251
| | | | | | | | | | | | Previously, when you created a module you had to specify the path. Now Module instances can use the module path to look up their paths, and there are methods for determining whether the module is present (if the path is present). Also cleaned up the methods for figuring out what's in the module (plugins, etc.). Signed-off-by: Luke Kanies <luke@madstop.com>
* Moving file-searching code out of Puppet::ModuleLuke Kanies2009-05-157-272/+296
| | | | | | | | | | The Module class had a bunch of code for finding manifests and templates even when not in a module, and it complicated the class unnecessarily. This moves that code to a new, hackish-but-sufficient module for just that purpose. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2234 - fixing all of the tests broken by my bindaddress fixLuke Kanies2009-05-157-11/+42
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2221 - pluginsignore should work againLuke Kanies2009-05-153-2/+26
| | | | | | | | We weren't splitting on whitespace, which is necessary since the settings don't support arrays but files expect them. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix snippets tests failing because of activated storeconfigsBrice Figureau2009-05-152-0/+25
| | | | | | | | | All the snippets tests were failing because some parser and scope tests activated storeconfigs without reseting the state. Activating storeconfigs is not undoable at the moment by just setting storeconfig=false as some terminus are changed. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix failing test: file.close! and file.path ordering fixBrice Figureau2009-05-151-1/+2
| | | | | | | | | | | On ruby 1.8.7 file.close! nils the internal file path. So the following pattern: file = temp file.close! file = file.path doesn't work. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Confine stomp tests to Stomp enabled systemsBrice Figureau2009-05-151-0/+4
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix some master failing testsBrice Figureau2009-05-153-3/+3
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix bug #2124 - ssh_authorized_key always changes target if target is not ↵Francois Deppierraz2009-05-152-1/+12
| | | | defined
* Fixing #2265 - rack is loaded with features rather than manuallyLuke Kanies2009-05-141-1/+0
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Added .git to pluginsignore default list of ignoresJames Turnbull2009-05-091-1/+1
|
* Cleanup of the Puppet Rakefile and removal of the requirement for the ↵James Turnbull2009-05-093-957/+53
| | | | Reductive Build Library
* Fix #1409 once again, including testFrancois Deppierraz2009-05-062-6/+23
| | | | | | | "user doesn't exit" error appeared once again after the changes which were applied in order to fix #2004. Validation must only check attributes presence, not their value.
* Added split functionJames Turnbull2009-05-061-0/+11
|