summaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* | Fixing #1913 - 'undef' resource values do not get copied to the dbLuke Kanies2009-02-061-1/+2
| | | | | | | | | | | | | | | | | | | | | | There are unfortunately no tests for this patch; the Rails code is disappointingly low on tests as it is, and it would have been essentially an herculian effort add all of the necessary tests just to make sure this worked. I've verified it works in practice, which should be sufficient for now. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing #1914 - 'undef' relationship metaparameters do not stackLuke Kanies2009-02-061-0/+1
| | | | | | | | | | | | | | | | This allows you to specify that a metaparameter is undef inside a definition and keep any parameters from being inherited from the parent. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Typo fixJames Turnbull2009-01-271-5/+5
| |
* | Fixed #1879 - Added to tidy documentationJames Turnbull2009-01-231-5/+15
| |
* | Fixed #1881 - Added md5lite explanationJames Turnbull2009-01-221-1/+5
| |
* | Fixed #1877 - Tidy type reference update for use of 0James Turnbull2009-01-221-1/+3
| |
* | Fix #1560Paul Lathrop2009-01-221-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds an rspec test which demonstrates #1560 and a custom 'process' method for the aliases provider to fix it. The default processing uses split() to break the line into records on the separator, which breaks if records can contain the separator. The custom method I've added uses a 'limited' split() to break the line on the first separator only. This commit fixes #1560 Signed-off-by: Paul Lathrop <paul@tertiusfamily.net>
* | In order for ReST formatting to work properly, newlines andJames Turnbull2009-01-221-1/+1
| | | | | | | | | | | | indentation of doc strings must be retained. Signed-off-by: Thomas Bellman <bellman@nsc.liu.se>
* | Fix Bug #1629Francois Deppierraz2009-01-131-12/+52
| | | | | | | | | | A refactoring of ssh_authorized_key parsed provider was needed and tests were improved. flush method has been split for clarity.
* | Fix #1835 : Add whitespace/quote parsing toBryan Kearney2009-01-131-18/+36
| |
* | Fix #1847 - Force re-examination of all files to generate correct indicesBrice Figureau2009-01-131-0/+1
| | | | | | | | | | | | | | This bug is in fact a RDoc issue. The work-around is to always force a full parse scan and not only what has changed. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #1829 - Add puppet function versioncmp to compare versionsBrice Figureau2008-12-271-0/+10
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #1828 - Scope.number? wasn't strict enough and could produce wrong resultsBrice Figureau2008-12-271-3/+3
| | | | | | | | | | | | | | Some invalid numbers were treated as numbers and conversion to Integer was failing returning 0 (for instance 0.24.7). Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fix #1807 - make Puppet::Util::Package.versioncmp a module functionBrice Figureau2008-12-271-0/+2
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Merge branch '0.24.x'James Turnbull2008-12-2731-183/+294
|\| | | | | | | | | Conflicts: CHANGELOG
| * Not using a temporary file when locking files for writing.Luke Kanies2008-12-151-9/+6
| | | | | | | | | | | | | | | | | | | | The temporary file was not actually useful, because we could never really get atomic renames, for annoying, complicated reasons. This hopefully finally fixes #1812. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Issue 1804 VDev with the same devices should be in syncAndrew Shafer2008-12-152-13/+50
| | | | | | | | | | Added VDev and MultiVDev properties to the ZPool type to handle logic Vdevs with the same devices are now in sync even if the order changes
| * Documentation fixesJames Turnbull2008-12-1324-130/+181
| |
| * Fixing #1812 (hopefully) - adding read and write locks to yaml.Luke Kanies2008-12-121-3/+12
| | | | | | | | | | | | | | | | It's obviously not really possible to test that this fixes it, but I'm confident that the locks work, and now we're using them, so it *should*. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Preparing to fix #1812 - Moving locking code to a moduleLuke Kanies2008-12-123-36/+53
| | | | | | | | | | | | | | This moves the locking code out of Puppet::Util into a separate module, to make the code cleaner. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fix #1815 - puppetdoc --all crash on resource overrideBrice Figureau2008-12-101-1/+1
| |
* | Fixing #1729 - puppetmasterd can now read certs at startupLuke Kanies2008-12-184-95/+83
| | | | | | | | | | | | | | | | | | | | The main aspect of this solution is to create a site-wide Puppet::SSL::Host instance to cache ssl key and certificate, so that by the time we've switched UIDs, we've got the key and cert in memory. Then webrick just uses that, rather than creating a new Host instance. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Canonicalizing Setting section names to symbols.Luke Kanies2008-12-181-14/+14
| | | | | | | | | | | | | | | | | | I thought I was already using symbols everywhere so it didn't matter, but there are a few places (e.g., the process name) where they were strings, and that made things not work so much. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing all of the test/ tests I broke in previous dev.Luke Kanies2008-12-185-21/+21
| | | | | | | | | | | | | | | | | | Most of these are straightforward changes to the tests, but a couple required small refactorings (e.g., References can now be created with Puppet::Type instances, and they know how to extract the type/title from them). Signed-off-by: Luke Kanies <luke@madstop.com>
* | Deprecating the Puppet::Type.create.Luke Kanies2008-12-1811-13/+15
| | | | | | | | | | | | | | This method is no longer necessary; you can use the normal 'new' class method. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Deprecating 'Puppet.type'; replacing all instances with Puppet::Type.typeLuke Kanies2008-12-1817-20/+22
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Finishing the work to use Puppet::Resource instead of TransObjectLuke Kanies2008-12-187-105/+90
| | | | | | | | | | | | | | | | | | | | | | | | This was a complicated project because TransObject had made its way into too many classes. The usage by Util::Settings was particularly nefarious. Refactoring and fixing this exposed some other issues. The main complication, though, was the extent to which the Puppet::Type class depended on TransObject. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding name/namevar abstraction to Puppet::Resource.Luke Kanies2008-12-181-7/+33
| | | | | | | | | | | | | | This hopefully provides a single place to manage this complexity, and I'll be using it to simplify Puppet::Type. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Supporting a nil expirer on cacher objects.Luke Kanies2008-12-181-1/+8
| | | | | | | | | | | | | | | | | | | | This works for those classes like Puppet::Type that use an expirer that isn't always present (they use their catalog as an expirer). The behaviour is now that if there is no expirer, expire() is a no-op and all data is considered expired all the time, so it's always fresh. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Setting resource defaults immediately.Luke Kanies2008-12-182-6/+10
| | | | | | | | | | | | | | This makes it easier for later parameters to depend on the values of earlier parameters. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Refactoring the Settings class to use Puppet::ResourceLuke Kanies2008-12-182-181/+69
| | | | | | | | | | | | | | It also now uses the Catalog instead of the recursive TransObject stuff. Signed-off-by: Luke Kanies <luke@madstop.com>
* | TransObject is nearly deprecated now.Luke Kanies2008-12-185-96/+71
| | | | | | | | | | | | | | | | | | This is all of the plumbing work, the only real thing left to do is to fix the Settings class so that it uses Puppet::Resource instances instead of TransObject and TransBucket. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Simplifying the initialization interface for ReferencesLuke Kanies2008-12-181-5/+10
| | | | | | | | | | | | | | | | | | You previously had to call new(nil, "Foo[bar]") if you just had the resource reference as a string. Now you can call new("Foo[bar]"), but the old behaviour works, too. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Replacing TransObject usage with Puppet::ResourceLuke Kanies2008-12-184-520/+95
| | | | | | | | | | | | | | | | | | This completely reorganizes how RAL resources are initialized, and in the process I was able to remove a lot of code (I removed other apparently obsolete code at the same time). Signed-off-by: Luke Kanies <luke@madstop.com>
* | Renaming the "Catalog#to_type" method to "Catalog#to_ral"Luke Kanies2008-12-183-9/+12
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Using Puppet::Resource to convert parser resources to RAL resourcesLuke Kanies2008-12-181-1/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding resource convertion to the parser resourcesLuke Kanies2008-12-181-44/+32
| | | | | | | | | | | | | | Also uses Puppet::Resource's method for creating transportable resources. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding equality testing to Puppet::Resource::ReferenceLuke Kanies2008-12-181-0/+6
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Renaming Puppet::Node::Catalog to Puppet::Resource::CatalogLuke Kanies2008-12-188-11/+11
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Renaming Puppet::ResourceReference to Puppet::Resource::ReferenceLuke Kanies2008-12-099-19/+20
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding Trans{Object,Bucket} backward compatibility to Puppet::ResourceLuke Kanies2008-12-092-0/+69
| | | | | | | | | | | | This is further progress toward #1808. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Starting on #1808 - Added a base resource class.Luke Kanies2008-12-092-5/+112
| | | | | | | | | | | | | | | | | | | | | | This class borrows heavily from the Puppet::Parser::Resource class and from Puppet::TransObject, partially because it will hopefully eventually supplant both of them. The class isn't used at all yet; now we need to go through the codebase and remove everything related to TransObject. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2008-12-0914-96/+614
|\| | | | | | | | | | | | | | | | | Conflicts: CHANGELOG lib/puppet/type/tidy.rb spec/unit/type/file/ensure.rb spec/unit/type/tidy.rb
| * Add a unique name to objects so we can determine uniqueness when read back inJohn Ferlito2008-12-092-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nagios object definitions have been updated to correlate with Nagios 3.0.6. In Nagios it is possible to have multiple service checks with the same service_description. eg I could have an check with a service_description of 'SSH' for multiple hosts. So in puppet we can't use it as a unique name for the resource. This patch modifies the code to use $name as the unique name. For some types eg command_name $name ends up in the config and thus we can tell which puppet resources match to which nagios ones. For other types like service there is no direct mapping from $name to a nagios attibute. So we use a custom attribute called _naginator_name. Signed-off-by: John Ferlito <johnf@inodes.org>
| * Fix the spec tests to work on other platforms, do the confine around OS X ↵Nigel Kersten2008-12-092-5/+9
| | | | | | | | versions more sanely
| * remove unnecessary mk_resource_methods callNigel Kersten2008-12-091-2/+0
| |
| * Add a unique name to objects so we can determine uniqueness when read back inJohn Ferlito2008-12-092-74/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nagios object definitions have been updated to correlate with Nagios 3.0.6. In Nagios it is possible to have multiple service checks with the same service_description. eg I could have an check with a service_description of 'SSH' for multiple hosts. So in puppet we can't use it as a unique name for the resource. This patch modifies the code to use $name as the unique name. For some types eg command_name $name ends up in the config and thus we can tell which puppet resources match to which nagios ones. For other types like service there is no direct mapping from $name to a nagios attibute. So we use a custom attribute called _naginator_name. Signed-off-by: John Ferlito <johnf@inodes.org>
| * Bug #1803 Zfs should auto require the ancestor file systemsAndrew Shafer2008-12-081-0/+6
| |
| * Refactor #1802 Use 'zfs get -H -o value' instead of parsing output for valueAndrew Shafer2008-12-081-12/+1
| | | | | | | | just simplifying code
| * Fixing #1800 - tidy now correctly ignores missing files and directoriesLuke Kanies2008-12-081-1/+8
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>