summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| | * Fixing #1991 - ldap booleans get converted to booleansLuke Kanies2009-03-052-0/+45
| | | | | | | | | | | | 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>
| | * dbfix - fix typo and close another possible inconsistencyBrice Figureau2009-03-041-3/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes a typo in one request producing a SQL error. It also close an issue where param_values could point to inexistant param_names after the deduplications of those. It does this by rewriting the param_values to point to the highest id of the param_names of the same name. The same operation is performed on the tags. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.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>
| | * Further fix to #1910James Turnbull2009-03-041-1/+1
| | |
| | * Fixing #2013 - prefetching had a mismatch between type and titleLuke Kanies2009-03-042-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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 to rake tests for reductivelabs buildJames Turnbull2009-03-041-3/+3
| | |
| | * Fixed report reference pageJames Turnbull2009-03-031-0/+1
| | |
| | * Fixing broken 0.24.x tests in test/.Luke Kanies2009-02-283-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These tests once again largely were caused by /usr/sbin not being in the path and by ~ not being writable. The only tests still failing are Rails tests, and my guess is that they're all failing because of the recent work by Brice. They should probably just be removed. Signed-off-by: Luke Kanies <luke@madstop.com>
| | * Fixing every failing test I can find on the build server.Luke Kanies2009-02-2814-7/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All but one of these tests is fixed by: * Stubbing Puppet.settings.use * Adding /usr/sbin to PATH The only other one was the package integration test, which stupidly assumed a default was specified in the test. The fix here is twofold: Remove that assumption (the test is now 'pending' if no default is available), and add a default for Ubuntu. (The default is in the test, not the default package provider - that is, it's testing the default package provider, thus it can't rely on that information.) Signed-off-by: Luke Kanies <luke@madstop.com>
| | * This script fixes the most common issues with inconsistentJames Turnbull2009-02-282-0/+100
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | storeconfigs database (including duplicate resources record, duplicate param_values records, dangling records...). Usage: stop all puppetmasters backup your database! % cat ext/dbfix.sql | mysql puppet relaunch all puppetmasters Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com> ---
| | * Make puppetd --waitforcert option behave as documented:Paul Lathrop2009-02-282-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | "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-2814-12/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-284-0/+122
| | | | | | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| | * Fixed #1963 - Failing to read /proc/mounts for selinux kills file downloadsJames Turnbull2009-02-282-1/+5
| | |
| | * Fixed #2025 - gentoo service provider handle only default init levelJames Turnbull2009-02-272-1/+3
| | |
| | * Fixed #1910 - updated logcheckJames Turnbull2009-02-262-1/+4
| | |
| | * 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>
* | | Adding #2122 - you can specify the node to test with puppet-testLuke Kanies2009-04-061-13/+21
|/ / | | | | | | | | | | | | | | This commat adds a --node option to puppet-test, so that you can use locally cached node and fact data for testing local and remote compiles. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fix #2107 - flatten resource references arrays properlyBrice Figureau2009-03-272-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resource parameters of the form [Res[a], Res[a,b]] ends being evaluated as [Res[a], [ Res[a], Res[b] ] This last form was not flattened when transfomed into RAL type, which in turn prevented the sub array to be converted in regular resource references. Thus the type was choking when encountering those native parser references instead of usual resource reference. The fix consists in flattening array of references before transformation to trans objects for RAL. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #2101 - Return to recurse=0 == no recursion behaviorBrice Figureau2009-03-271-1/+10
| | | | | | | | | | | | | | | | | | Ticket #1469 introduced an incorrect change of behaviour where recurse = 0 was considered as an infinite recursion, unlike before. This patch restores this behavior, and makes some test pass. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #2101 - fix failing testBrice Figureau2009-03-271-1/+5
| | | | | | | | | | | | | | | | This code hasn't been modified since the introduction of the fileset recurselimit parameter. Tests depending on this code were failing. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #2101 - fix recurselimit == 0 bad behaviourBrice Figureau2009-03-272-5/+5
| | | | | | | | | | | | | | After the fix for #1469, recurselimit = 0 was considered as an infinite recursion which is the reverse of what it was before. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Merge branch 'tickets/master/2110'James Turnbull2009-03-271-1/+1
|\ \
| * | Fixed #2110 - versioncmp brokenJames Turnbull2009-03-261-1/+1
| | |
* | | Added rake ci:all taskJames Turnbull2009-03-261-0/+3
|/ /
* | CHANGELOG updatesJames Turnbull2009-03-211-0/+4
| |
* | Moved of puppetd, puppetca, puppetmasterd, puppetrun binary from bin to sbinJames Turnbull2009-03-215-4/+12
| |
* | Fixed #2086 - Fixes to make building tarballs easierJames Turnbull2009-03-211-2/+4
| |
* | Fix #1469 - Add an option to recurse only on remote sideBrice Figureau2009-03-205-45/+107
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using recurse and a source, if the client side has many files it can take a lot of CPU/memory to checksum the whole client hierarchy. The idea is that it is not necessary to recurse on the client side if all we want is to manage the files that are sourced from the server. This changeset adds the "remote" recurse value which prevents recursing on the client side when a source is present. Since it also is necessary to limit the remote side recursion a new File{} parameter has been added called "recurselimit". Moreover, the Filetset API is changing to allow the new recurselimit parameter, and passing the recursion depth limit in the recurse parameter as an integer is now deprecated and not supported anymore. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Forbidding REST clients to set the node or IPLuke Kanies2009-03-203-1/+27
| | | | | | | | | | | | | | | | This is done for security reasons - if a client is unauthenticated, we don't want them to be able to just configure their own authentication information. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing #1557 - Environments are now in REST URIsLuke Kanies2009-03-2022-185/+88
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit includes multiple, related changes, all in one commit because the whole thing was necessary to reach a functional tree again: * The URI starts with the environment, so: /production/certificate/foo /development/file_content/path/to/your/file * All REST handling is done by a single instance mounted at / for webrick and Mongrel, rather than having individual instances mounted at, say, /certificate. * All REST URI translation is done by an API module. Currently only the 'v1' module exists with no support for additional modules, but it's well-separated and will be easy to expand as we need it. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding explicit optional attribute to indirection requestsLuke Kanies2009-03-202-9/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | Previously, any option that had a setter was treated as an attribute, but now we're specifying the list of attributes settable via options. We also have a to_hash method that will take all of the options and all of those attributes and join them back into a hash. This method is used by the REST Handler module, since it uses the indirection request internally. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Removing an unused source fileLuke Kanies2009-03-201-59/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding a model accessor to the Request classLuke Kanies2009-03-202-4/+27
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Requests now use default environment when none is specifiedLuke Kanies2009-03-202-1/+12
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Moving the REST API functions into a moduleLuke Kanies2009-03-207-169/+197
| | | | | | | | | | | | | | | | | | | | | | | | | | This module is now used by the client and server side, rather than having a Handler module that's 90% server functionality but also used by the client. While we don't automatically get api choice from this, it at least provides a pattern for how we'll handle API development over time. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Using the Handler for the REST api on both sides of the connectionLuke Kanies2009-03-204-296/+124
| | | | | | | | | | | | | | | | | | | | Things are actually in a broken state here because we've got a conflict between how the two sides do their work and some extraction needs to get done. This commit is just a stopping-point so I can do the necessary refactoring. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding REST::Handler methods for converting between indirection and urisLuke Kanies2009-03-204-0/+155
| | | | | | | | | | | | | | | | | | | | This is the first main phase to having a common module for handling the REST api - this Handler module will be included by all of the web server REST modules and the Indirector Request class, so there's a common place that understands how the URI consists. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding environment support to the REST URILuke Kanies2009-03-204-17/+56
| | | | | | | | | | | | Also adding it to the Indirection Request. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Moving the query_string method to RequestLuke Kanies2009-03-204-100/+104
| | | | | | | | | | | | | | | | It required a request instance and didn't use the REST class it was in, so it makes more sense in the Request class. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding tests for the REST query string usageLuke Kanies2009-03-201-5/+21
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing #2108 - pi should work againLuke Kanies2009-03-201-6/+8
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fix #1088 - part2 - Add rspec testsBrice Figureau2009-03-144-37/+216
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #1088 - Collections overridesBrice Figureau2009-03-144-721/+860
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset defines a new syntax to override collection of resources (virtual or not). This feature is not constrained to the override in inherited context as usual resource override. The collection of resource supports a query like regular collection of virtual or exported resources. Usage example: file { "/tmp/testing": content => "whatever" } File<| |> { mode => 0600 } It also introduces a different behaviour for collection of catalog resources. Before this patch, only virtual resources were collected, now all resources (virtual or no) are collected and can be overriden. That means it is now possible to do: File <| |> { mode => 0600 } And all the Files resources will have mode 0600. It is then possible to have this puppet pattern: file { "/tmp/a": content => "a" } file { "/tmp/b": content => "b" } File <| title != "/tmp/a" |> { require => File["/tmp/b"] } which means that every File requires a file. Moreover it is now possible to define resource overriding without respecting the override on inheritance rule: class a { file { "/tmp/testing": content => "whatever" } } class b { include a File<| |> { mode => 0600 } } include b Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fixed #2071 - Updated LDAP schemaJames Turnbull2009-03-142-1/+4
| |
* | Fixing tests broken in previous commitsLuke Kanies2009-03-112-14/+5
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
generated by cgit (git 2.34.1) at 2025-09-30 19:51:37 +0000