summaryrefslogtreecommitdiffstats
path: root/lib/puppet
Commit message (Collapse)AuthorAgeFilesLines
...
* Changed tabs to spaces without interfering with indentation or alignmentIan Taylor2009-06-0622-255/+255
|
* Fix #2308 - Mongrel should use X-Forwarded-ForBrice Figureau2009-06-052-2/+2
| | | | | | | | | | | | Mongrel puppet code uses REMOTE_ADDR to set the ip address which will be use to authenticate the client access. Since mongrel is always used in a proxy mode with Puppet, REMOTE_ADDR is always the address of the proxy (usually 127.0.0.1), which defeats the purpose. With this changeset, the mongrel code now uses the X-Forwarded-For HTTP header value if it is passed over the REMOTE_ADDR. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixes Bug #2324 - Puppetd fails to start without railsStéphan Gorget2009-06-051-1/+1
| | | | Signed-off-by: Stéphan Gorget <gorget@ocre.cea.fr>
* Enhance versioncmp documentationBrice Figureau2009-06-041-1/+22
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* * provider/augeas: allow escaped whitespace and brackets in pathsDavid Lutterkort2009-06-031-1/+1
|
* * provider/augeas: match comparison uses '==' and '!=' againDavid Lutterkort2009-06-032-7/+7
| | | | | | Change the syntax for match in an onlyif back to using '==' for equality comparison instead of 'eq' (this was the behavior before 9a2642) and make operator for inequality '!='.
* * provider/augeas (process_match): no match results in empty arrayDavid Lutterkort2009-06-031-1/+1
|
* * provider/augeas: remove useless checks for nilDavid Lutterkort2009-06-031-36/+32
|
* * provider/augeas: simplify evaluation in process_get/matchDavid Lutterkort2009-06-031-10/+9
|
* * provider/augeas (open_augeas): use Augeas flag names, not intsDavid Lutterkort2009-06-031-2/+2
|
* * provider/augeas: ensure Augeas connection is always closedDavid Lutterkort2009-06-031-35/+41
| | | | | | | | | | | | | | | | | | | | Turn code like open_augeas ... close_augeas into begin open_augeas ... ensure close_augeas end Would have liked to pass the '...' as a block into a 'with_augeas' method, but that makes the spec tests fail in ways I don't understand.
* * provider/augeas: minor code cleanupDavid Lutterkort2009-06-031-44/+44
| | | | | | | - remove '()' for empty method calls - remove 'self.' for calling methods on self - use the fact that nil is false - remove redundant parens around some expressions
* * provider/augeas (parse_commands): use split to split string into linesDavid Lutterkort2009-06-031-3/+1
|
* * provider/augeas: remove trailing whitespace (no functional change)David Lutterkort2009-06-031-4/+4
|
* Brought in lutters parse_commands patch and integrated it into the type.David Lutterkort2009-06-032-63/+113
| | | | | | | | | | | | | | | | | | | | | This includes reworking the get and match commands as well. This change introduces a few small changes. These are: 1) There can be no ' or " characters around path elements. 2) The context is now only prepended to relative paths. So, if a path is specified as /foo/bar the context will not be appended. It will only be appended if it is foo/bar 3) The syntax for array matching in the onlyif is now eq or noteq. It was == before. 4) The get and set commnands used to concatenate all items at the end of a string so "set path to some value" would be interpreted as "set" "path" "to some value". This is no longer supported. The caller must put ' or " around the "to some value" for it to work" Cherry-pick of 9a2642 from 0.24.x Conflicts: lib/puppet/provider/augeas/augeas.rb
* Fixing a bunch of warningsLuke Kanies2009-06-039-15/+15
| | | | | | This commit should have no functional effect. Signed-off-by: Luke Kanies <luke@madstop.com>
* Always providing a value for 'exported' on Rails resourcesLuke Kanies2009-06-031-2/+11
| | | | | | | | | | | We often didn't set a value, unless it was true, which meant that if it had previously been true but was now false, we didn't fix it. We also were not always saving modified resources, which in some cases resulted in work not getting saved. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2230 - exported resources work againLuke Kanies2009-06-031-0/+1
| | | | | | | | This somehow got lost in the conversion from Parser resources to Puppet resources. We now copy over the 'exported' value. Signed-off-by: Luke Kanies <luke@madstop.com>
* Disabling the catalog cache, so puppetqd is compatible with storeconfigsLuke Kanies2009-06-031-0/+6
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing the rails feature to be compatible with 2.1+Luke Kanies2009-06-031-19/+16
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Using Message acknowledgement in queueingLuke Kanies2009-06-031-1/+4
| | | | | | | | | This allows a more fine-grained load-balancing of the queue, which makes it easy to spin up multiple puppetqd instances and process the queues faster. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2315 - ca --generate works againLuke Kanies2009-06-031-9/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix #2220 - Make sure stat is refreshed while managing FilesBrice Figureau2009-06-021-1/+1
| | | | | | | | | | | Since the introduction of the attribute cacher, Files couldn't refresh their 'stat' attribute while applying the properties. This could led to misunderstanding (like not setting mode) because the stat didn't reflect the physical state of the entity, especially when creating directories, where puppet was taking decision thinking the directory didn't exist although it was just created. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* JRuby OpenSSL implementation is more strict than real ruby one andJames Turnbull2009-05-301-1/+1
| | | | requires certificate serial number to be strictly positive.
* Logging when a cached catalog is used.Luke Kanies2009-05-281-0/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Changing Puppet::Cacher::Expirer#expired? method nameLuke Kanies2009-05-283-3/+4
| | | | | | | | | This method name clashed with Puppet::Indirector::Envelope#expired?, and its name wasn't actually very appropriate. The new method name is 'dependent_data_expired?'. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2240 - external node failures now log outputLuke Kanies2009-05-281-5/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2237 - client_yaml dir is always created by puppetdLuke Kanies2009-05-281-0/+2
| | | | | | | This is just a simple call to Puppet.use in the puppetd setup method. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2228 - --logdest works again in puppetd and puppetmasterdLuke Kanies2009-05-272-2/+2
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing the Agent so puppetrun actually works server-sideLuke Kanies2009-05-271-2/+2
| | | | | | | 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-271-1/+1
| | | | | | | 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-271-1/+4
| | | | | | | | 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 #2280 - Detailed exit codes fixJames Turnbull2009-05-271-1/+1
|
* Fixing #2253 - pluginsync failures propagate correctlyLuke Kanies2009-05-261-8/+8
| | | | | | | | | | | 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>
* Refactoring resource generation slightlyLuke Kanies2009-05-261-5/+4
| | | | | | | | | | | 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>
* 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-261-0/+1
| | | | | | | 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
|
* 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-221-0/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix for #2234: test fails with old Rack versionChristian Hofstaedtler2009-05-202-2/+24
|
* Fixing #2273 - file purging works more intuitivelyLuke Kanies2009-05-202-14/+13
| | | | | | | | | | | | | | 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-202-2/+18
| | | | | | | | 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-202-1/+101
| | | | | | | | | | | 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-201-17/+22
| | | | | | | | | 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-201-2/+18
| | | | | | | | | | | 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-201-4/+34
| | | | | | | | | 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-201-0/+3
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Protecting Stomp client against internal failuresLuke Kanies2009-05-181-1/+5
| | | | | | | | 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>