summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Update man pages and partial doc fix to #3491James Turnbull2010-04-128-234/+199
| |
| * Fixed #3532 - Typo in lib/puppet/ssl/host.rbJames Turnbull2010-04-121-1/+1
| | | | | | | | Thanks to Jasper Lievisse Adriaanse for the fix.
| * Fix #3496 - suppress transaction debug messageJames Turnbull2010-04-051-1/+3
| | | | | | | | Fix via Brice Figureau
| * Updated version and CHANGELOG to 0.25.5rc1James Turnbull2010-04-013-25/+640
| |
| * Modify SuSE spec file for 0.25.x and correct shebang lines forAndrew Forgue2010-04-012-12/+30
| | | | | | | | puppetd/puppetmasterd
| * Fixes #3460 - Makes Puppet FHS compliant by moving /var/puppet to ↵James Turnbull2010-04-0110-22/+14
| | | | | | | | /var/lib/puppet
| * Fix for #3101 (bug in MRI 1.8.7)Markus Roberts2010-03-311-1/+12
| | | | | | | | | | | | | | | | | | | | | | Due to a bug in Ruby 1.8.7 net/http will attempt to close a connection that wasn't successfully opened (it's nil), first checking to see if the connection is already close, and thus raising a method missing exception. This bug causes error messages that are confusing / misleading. To get around this, we add a closed? method to nil such that a nil (unopened) connection is always considered closed, allowing the real problem to be reported.
| * Fixes #3419. OS X 10.6 Ruby doesn't set supplementary groupsNigel Kersten2010-03-311-2/+26
| |
| * Revert the guts of #2890Markus Roberts2010-03-316-77/+112
| | | | | | | | | | | | | | | | | | | | This patch reverts the semantically significant parts of #2890 due to the issues discussed on #3360 (security concerns when used with autosign, inconsistency between REST & XMLRPC semantics) but leaves the semantically neutral changes (code cleanup, added tests) in place. This patch is intended for 0.25.x, but may also be applied as a step in the resolution of #3450 (refactored #2890, add "remove_certs" flag) in Rolwf.
| * Fail gracefully on packages that don't have the HOMEPAGE variable set (e.g. ↵Gábor Vészi2010-03-311-1/+1
| | | | | | | | dev-lang/php).
| * Fixed #3443 - Typo in mount typeJames Turnbull2010-03-281-2/+3
| |
| * Fixes #3135 - darwin doesn't support 'mount -o remount'James Turnbull2010-03-271-1/+2
| | | | | | | | Thanks for Eric Sorenson for the patch.
| * Adding :catalog_terminus settingLuke Kanies2010-03-277-18/+50
| | | | | | | | | | | | | | | | | | | | This will rarely be used, but it enables even more architectural flexibility, such as precompiling catalogs and storing them in memcached or equivalent. With this setup, a single host can probably serve all catalogs and you would then just have as many compiling hosts as needed. Signed-off-by: Luke Kanies <luke@reductivelabs.com>
| * fixing obsolete comment in puppetdLuke Kanies2010-03-271-3/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@reductivelabs.com>
| * Adding support for only using cached catalogsLuke Kanies2010-03-273-2/+26
| | | | | | | | | | | | | | | | | | This basically allows a sysadmin to control when a client will compile a new catalog - with this option enabled, the client will use the cached catalog as long as it has one, only recompiling when run with the option disabled. Signed-off-by: Luke Kanies <luke@reductivelabs.com>
| * Refactoring Configurer to enable the next featureLuke Kanies2010-03-271-27/+30
| | | | | | | | Signed-off-by: Luke Kanies <luke@reductivelabs.com>
| * Fix for #3366 - --tags '' treated as boolean 'true'Markus Roberts2010-03-262-4/+21
| | | | | | | | | | This is the patch from Mike Pountney <Mike.Pountney@gmail.com> off the list with the additional test Luke requested.
| * Supressing warnings (not really failures) in test/unitJesse Wolfe2010-03-262-9/+19
| |
| * Fix test using wrong Puppet util filesetting groupRein Henrichs2010-03-261-1/+1
| |
| * Mock user in SUIDManager testsRein Henrichs2010-03-261-2/+7
| |
| * Removing resources generate testsRein Henrichs2010-03-261-78/+0
| | | | | | | | | | | | Tests that generating resources performs a check and only returns resources that check as true. There is already spec coverage for this behavior.
| * Removing old test for service/debian providerRein Henrichs2010-03-261-58/+0
| | | | | | | | it has been superceded by an rspec spec.
| * Replace test/unit file write test with specRein Henrichs2010-03-262-32/+36
| |
| * Fix for #3424 and tests to prove it.Markus Roberts2010-03-252-4/+44
| | | | | | | | | | | | | | | | The original pure ruby yaml patch missed some edge cases; specifically, classes that were modified by the syck version to directly call it and thus never reached the pure ruby version. This adds monkey patches to all of those case which we might reasonably care about (omitting, for example, calls within the syck version to itself) and tests which show that the monkey patch works.
| * Fixed changelog Rake taskJames Turnbull2010-03-251-1/+1
| |
| * Fix #3155 - prevent error when using two matching regex in cascadeBrice Figureau2010-03-258-20/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following manifest: case $var { /match/: { if $var =~ /matchagain/ { } } } is failing because the "=~" operators when matching sets an ephemeral variable in the scope. But the case regex also did it, and since they both belong to the same scope, and Puppet variables are immutables, the scope raises an error. This patch fixes this issue by adding to the current scope a stack of ephemeral symbol tables. Each new match operator or case/selector with regex adds a new scope. When we get out of the case/if/selector structure the scope is reset to the ephemeral level we were when entering it. This way the following manifest produces the correct output: case $var { /match(rematch)/: { notice("1. \$0 = $0, \$1 = $1") if $var =~ /matchagain/ { notice("2. \$0 = $0, \$1 = $1") } notice("3. \$0 = $0, \$1 = $1") } } notice("4. \$0 = $0") And the output is: 1. $0 = match, $1 = rematch 2. $0 = matchagain, $1 = rematch 3. $0 = match, $1 = rematch 4. $0 = Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixing #3148 Settings#without_noop when run with no noop settingLuke Kanies2010-03-252-2/+32
| | | | | | | | | | | | | | Some tests didn't define this setting which caused this method to fail. Signed-off-by: Luke Kanies <luke@reductivelabs.com>
| * Another trivial follow-up fix for #2604: invalid path to zaml.rbMarc Fournier2010-03-251-1/+1
| | | | | | | | Signed-off-by: Marc Fournier <marc.fournier@camptocamp.com>
| * Fix inefficient SimpleGraph#matching_edgeBrice Figureau2010-03-253-20/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method has two issues: * it is inefficient when there are many events * it tries to match edges that shouldn't be matched With recursive file resources, many change events can be generated. The method used to find the good ones is pretty inefficient, allocating arrays and/or appending to arrays which is a slow operation that can consume lot of memory. Still with recursife file resources, the current code tries to match the events with edges pointing to generated sub-file-resources, which is not necessary. In fact this all happens because we masquerade the sub-generated resources with the topmost resource whic itself has auto-required links to them. There is no reason to send back those events to where they were generated. This patch tries to minimize allocations or array appending, it also collect event names (instead of events themselve) while matching since there are great chances there are way less events names than events (and we're matchin by name). This patch also makes sure we select only edges that don't point back to the event sources. Results for matching 1100 events: * old code: 22s * new code: 0.02s This patch also helps on the memory consumption side since the GC has almost no work to perform. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #3229 - use original value in case/selector regex matchingBrice Figureau2010-03-256-25/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The issue is that case/selectors are downcasing the value before it is compared to the options. Unfortunately regex are matching in a case sensitive way, which would make the following manifest fail: $var = "CaseSensitive" case $var { /CaseSensitive/: { notice("worked") } default: { fail "miserably" } } This patch fixes the issue by making sure the regexp match is done one the original (not downcased) value, but still doing a case sensitive match. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #2929 - Allow checksum to be "none"Brice Figureau2010-03-2510-5/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | File checksum is "md5" by default. When managing local files (not sourced or content) it might be desirable to not checksum files, especially when managing deep hierarchies containing many files. This patch allows to write such manifests: file { "/path/to/deep/hierarchy": owner => brice, recurse => true, checksum => none } Then puppet(d) won't checksum those files, just manage their ownership. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixed puppetlast typoJames Turnbull2010-03-241-1/+1
| | | | | | | | Patch thanks for Micah Anderson
| * Follow up for #2604, debug msg left behind.Marc Fournier2010-03-241-1/+0
| | | | | | | | Signed-off-by: Marc Fournier <marc.fournier@camptocamp.com>
| * Fix for #2604 Pure Ruby yaml generationMarkus Roberts2010-03-242-0/+343
| | | | | | | | | | | | | | | | This patch brings in a pure ruby yaml generation library, analagous to what we did with JSON/PSON, but without the renaming dodge we had to do in that case to avoid fighting with Rails. Signed-off-by: Markus Roberts <Markus@reality.com>
| * Fixes #3113 - When importing a manifest puppet needs to chillJames Turnbull2010-03-241-1/+4
| |
| * Fixes #3387 - Handle path elements with ticks and spacesBryan Kearney2010-03-243-4/+24
| | | | | | | | Unit tests for path changes
| * Fix for #3412 install.rb should not put "." first in the tmp_dirsMartin Englund2010-03-241-1/+1
| | | | | | | | Signed-off-by: Martin Englund <martin@englund.nu>
| * Fix #3186 - require function set relationship only on the last classBrice Figureau2010-03-243-4/+31
| | | | | | | | | | | | | | | | Due to the fact that resource.set_parameter is overwriting the previous set_parameters, we were losing the previous relationships we set there, either in a previous call of require or in the same call. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixed the return types were valid, and removed the copy paste error with the ↵Bryan Kearney2010-03-172-7/+7
| | | | | | | | exception logic
| * Fixing #3185 Rakefile is loading puppet.rb twiceJesse Wolfe2010-02-171-1/+1
| | | | | | | | | | | | | | A 'require' statement with a path confused ruby enough to cause the same file to get interpreted twice. Signed-off-by: Jesse Wolfe <jes5199@gmail.com>
| * Fix #3150 - require function doesn't like ::class syntaxBrice Figureau2010-02-142-0/+18
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Added time module to tagmail reportJames Turnbull2010-02-091-0/+1
| |
* | Updated CHANGELOG for 2.6.0rc2James Turnbull2010-07-121-0/+10
| |
* | [#4209] catalog.resources should return resourcesJesse Wolfe2010-07-111-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | type/user.rb was assuming that catalog.resources would return resources. In 0.25.x, it (confusingly) returns strings. type/user ignored this, and the codepath was a no-op. In 2.6, the method was renamed to something less confusing, causing the codepath in type/user to fail outright. This patch added a catalog.resources method that returns resource objects. As a side effect, user resources will now autorequire group resources again.
* | Fix for #4210 -- missing require in CAMarkus Roberts2010-07-111-1/+1
| | | | | | | | Added the require.
* | Minimal fix for #4205 -- incorrect Import loop messagesMarkus Roberts2010-07-111-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the narrow problem of #4205, wherein type_loader would reparse a file each time it was imported (causing the parser to incorrectly think that it was in a loop) by checking @imported inside the existing check on the thread-guarded @loaded. (@imported was being set but never checked). This works (and is thread safe) because all of this is going on inside a giant synchronize care of @loaded. But it, like the fix for #4208, does nothing about the global lock. Areas for future research: 1) Why is the looping inside of import? 2) Why are there separate @loaded and @imported tables? 3) Why is the parsing treated like a function (called deep in the structure) yet coded like a thread-savvy pseudo state monad (e.g. raising errors that presume it knows/owns what's going on outside the whole process)? These and many other exciting questions are deferred to #4211
* | Fix #4206 - import "path/*" tries to import files twiceBrice Figureau2010-07-112-1/+7
| | | | | | | | | | | | | | | | | | Due to the glob pattern used, we are trying to import manifests twice. Since it isn't possible (see #4205), it is not possible to use a pattern in an import statement. This patch makes sure manifests are returned only once. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Alt fix for #4207 -- serialize environments as their namesMarkus Roberts2010-07-111-0/+7
| | | | | | | | | | | | | | | | | | | | Environments contain a deal of transitory information and references to other objects, none of which is wanted when they are serialized. Rather than having this serialization concern propogate through the code by replacing environments by their names prior to serialization (which would be one way to address the problem) this patch changes environments so they only serialize their identity (name) and not their contents.
* | [#4208] Missing parameter breaks multithread compilationJesse Wolfe2010-07-101-3/+3
| | | | | | | | | | | | | | import_if_possible calls itself recursively, but it was failing to pass its block parameter to its younger self. Thus when the inner call reached the un-blocked case, it raised an exception rather than doing something useful.
* | Updated CHANGELOG for 2.6.0rc1James Turnbull2010-07-101-0/+622
| |
generated by cgit (git 2.34.1) at 2025-09-26 22:43:34 +0000