summaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* [#3172] puppet kick can take hostnames as bare argumentsJesse Wolfe2010-07-011-4/+8
|
* [#4108] Missing constants fail deliberately and with a messageNick Lewis2010-07-011-1/+6
| | | | | | | Previously, any failed call to Puppet::Application.find would result in an error being raised by const_get, resulting in a messy crash with a stack trace. Now that error is handled, and the application will print a message and exit.
* [#4092] Changed notify message to defined instead of changedNick Lewis2010-07-011-1/+1
| | | | | | Notify was returning a nil current value, rather than :absent, which caused puppet to think the message had been changed from blank to its new value, rather than defined as that value.
* Fix for #4091 -- require loop in executablesMarkus Roberts2010-07-012-2/+0
| | | | | The describe and resource (ralsh) applications required puppet, creating a loop & thus crashing them.
* [#4090] Change how RunMode instances are created so that an object for each ↵Matt Robinson2010-07-013-2/+8
| | | | | | | RunMode is only created once instead of every time it's called Got lots of unpredictable test failures, presumably because a new RunMode was being created every time we accessed the RunMode.
* [#4090] Fix last few tests and renames of mode to run_modeMatt Robinson2010-07-012-2/+2
|
* [#4090] Git rid of the idea that run_mode is a configurable option with defaultsMatt Robinson2010-07-015-9/+9
| | | | | Along the way this fixes an issue with 2.6 alpha that sections of the puppet config file were getting ignored.
* [#4090] Rename mode (:master, :agent, :user) to run_modeMatt Robinson2010-07-0114-34/+34
| | | | | | | | Mode is a terribly overused word. Files use it, puppetdoc uses it, and certs use it, and those are just the places that I happened to stumble upon. It makes reading code very confusing and finding things in code difficult. I know namespacing allows us to reuse words for method and variable names, but that doesn't mean we should.
* [#4089] Replace internal usage of :check with :auditNick Lewis2010-07-018-11/+11
| | | | | | Per Luke's replacement of :check with :audit, and deprecation of :check, I've replaced all of our internal uses of :check with :audit. Importantly, this silence the deprecation warnings during regular usage from eg. ralsh.
* [#3961] puppet cert --generate implies ca_location = :localJesse Wolfe2010-06-301-1/+5
|
* [#3961] Rename cert's @mode to @cert_mode to reduce confusionJesse Wolfe2010-06-301-5/+5
|
* maint: Confine a test that depends on sqliteMatt Robinson2010-06-281-0/+3
| | | | I'm also going to update the wiki to mention sqlite is a test dependency
* Removing obsolete nodescope conceptLuke Kanies2010-06-281-3/+1
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fix #3665 - part 2, node inheritance fixesBrice Figureau2010-06-281-5/+6
| | | | | | | | We were looking only to the class hierarchies when trying to find an ancestor to the current type. Thus we were never trying to climb up the hierarchy of nodes when evaluating nodes. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #3665 - main class shouldn't be a subscope of itselfBrice Figureau2010-06-281-1/+1
| | | | | | | | During the refactoring of AST hostclass/node to non AST objects, we lost the fact that the main class already comes with a scope (ie the top one), so when we evaluate its code we shouldn't create a subscope for it. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix for pre 1.8.7 compatibility in namvar patchMarkus Roberts2010-06-281-5/+1
| | | | | | | | There was a subtle 1.8.7 dependence in the composite key / namevar patch; Nick discovered that our assumtion that hashes could be used as hash keys does not hold in earlier bersions of ruby. This patch replaces the hash valued uniqueness_key with an array of the values of the (ordered) key attributes.
* Breaking require-loopMarkus Roberts2010-06-271-1/+0
| | | | | | Merging some of the branches creates a require-loop; this commit breaks it by removing the most recently added link (which I believe to have been unneeded / unrelated to the patch in which it snuck in).
* Fix for #3985 typo causing warningMarkus Roberts2010-06-261-1/+1
| | | | There was a comma missing from a raise, which caused a warning message on each run.
* Fix: puppet apply trying to use XMLRPC constantJesse Wolfe2010-06-251-5/+1
|
* Restore error handling for value=Jesse Wolfe2010-06-251-10/+11
|
* Fix tests broken by level-violation fixJesse Wolfe2010-06-251-4/+5
|
* Simplify the newattr method by removing a level violationJesse Wolfe2010-06-251-6/+0
| | | | and update the specs that were testing it.
* [#3139] Fixed a problem with the value method for parametersNick Lewis2010-06-251-1/+1
| | | | | The value method was failing to return any value in the case where the value was false.
* Fixed require warning documentationJames Turnbull2010-06-241-3/+2
|
* Fixed mcx documentation errorJames Turnbull2010-06-241-1/+1
|
* Documentation fixesJames Turnbull2010-06-241-8/+8
|
* Bug 3731. Applied Fix suggested by Doug Warner to always flatten out the arrayBryan Kearney2010-06-241-0/+1
|
* [#3810] Do not create a reports settings blockRein Henrichs2010-06-241-4/+2
| | | | | Puts reportdir and reporturl back in the "main" block because this makes tests break for reasons I don't understand.
* Tweak for fix for #1175 to fix test failuresMarkus Roberts2010-06-241-1/+1
| | | | | | Tests that weren't managing the environment but were still expecting to have functions defined in it were appalled when the functions/environments binding actually started working. This patch fixes those tests.
* [#2646] Fixes the change to onetime made in b96cd6cNick Lewis2010-06-241-1/+3
| | | | | | This change allows the --onetime command line option to be set by the Puppet[:onetime] global option or read from the config file, rather than forcing it to be read from the config file.
* [#3139] Make newattr idempotentJesse Wolfe2010-06-242-14/+6
| | | | | audit's munge was not idempotent because newattr was not idempotent. This patch simplifies newattr such that it becomes idempotent.
* Fix #3932 - Add --charset to puppetdoc for RDoc modeBrice Figureau2010-02-172-2/+4
| | | | | | | This adds the --charset option to puppetdoc for RDoc mode. This allows to set the charset for the generated html. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #3907 - Hash couldn't be initialized with an empty hashBrice Figureau2010-02-171-0/+5
| | | | | | | | | | The following manifest was failing: $data = {} This patch makes sure we initalize our ast hash with an empty ruby hash when it is created without any values. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #3871 - Add the 'in' operatorBrice Figureau2010-02-175-1014/+1066
| | | | | | | | | | | | | | | | | | | | | | | This operator allows to find if the left operand is in the right one. The left operand must be resort to a string, but the right operand can be: * a string * an array * a hash (the search is done on the keys) This syntax can be used in any place where an expression is supported. Syntax: $eatme = 'eat' if $eatme in ['ate', 'eat'] { ... } $value = 'beat generation' if 'eat' in $value { notice("on the road") } Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* [#3865] External subcommandsJesse Wolfe2010-02-171-1/+14
| | | | | This patch allows the puppet single-executable to invoke external, hyphenated subcommands, much like how git does.
* [#3802] Replace rug with zypperMatt Robinson2010-02-171-4/+4
| | | | | | OpenSuSE replaced rug with zypper so our code should too Signed-off-by: Matt Robinson <matt@puppetlabs.com>
* [#3766] Remove YAML fixupsJesse Wolfe2010-02-171-34/+7
| | | | | Remove workarounds that were only needed because ruby's builtin YAML lib is broken.
* Provides #3723. Add ability for execs to have several attempts at a successfulNigel Kersten2010-02-171-4/+58
| | | | | | | | | execution and fix minor bug with logoutput and returns as an array.. * Add 'tries' and 'try_sleep' parameters * Fix bug where returns is specified as an array and logoutput on * failure. * unit tests for both cases above.
* [#3674] Make sure that failing to load a feature isn't fatalJesse Wolfe2010-02-171-1/+1
|
* [#3674] Part 2: Autoloader load method should propagate failuresJesse Wolfe2010-02-172-19/+2
| | | | | | | | | Change Autoloader's load to re-raise exceptions that happen when trying to load files, rather than just warning. This version still does not raise an error if the file is not found, as doing so would change the behavior of 'load' pretty significantly, but I am ambivalent this.
* [#3674] Autoloader should propagate failuresJesse Wolfe2010-02-171-1/+1
| | | | | | | Change Autoloader's loadall to re-raise exceptions that happen when trying to load files, rather than just warning. Signed-off-by: Jesse Wolfe <jes5199@gmail.com>
* Fix #3667 - Fix class namespaceBrice Figureau2010-02-171-1/+5
| | | | | | | | Class namespace is different than namespace of nodes and definition as it contains the whole qualified name. This bit was left out in the type are not AST anymore refactoring. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Removing obsolete nodescope conceptLuke Kanies2010-02-171-16/+1
| | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing #3651 failing to pop comment stack for some constructsMarkus Roberts2010-02-172-121/+125
| | | | | Puppetdoc got confused because it wasn't popping the comment context for collections and resource defaults. This commit adds the popping.
* Fixing #3072 - Resource generation is recursiveLuke Kanies2010-02-171-0/+3
| | | | | | | | | | | | This allows you to create builtin nested resource types that generate other resources that generate other resources ad naseum. The primary point of this feature is that you can make builtin resource types that have a lot of the same encapsulation abilities of defined resource types. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixes #2646. Move onetime option from the agent to global defaults so we can ↵Nigel Kersten2010-02-172-6/+8
| | | | | | specify it in the config file.
* [#2522] authorized keys owner is verifiedMatt Robinson2010-02-171-21/+8
| | | | | | | | | | | | | The user method on the provider always returned what the resource should be, not what it actually was, so it always seemed to be insync to puppet. Also cleaned up some cruft on the perms that did different things depending on whether a user was specified on the resource. This isn't necessary since a user is required on the resource. Paired with: Jesse Wolfe Signed-off-by: Matt Robinson <matt@puppetlabs.com>
* Fixing #2337 - Adding 'freeze_main' settingLuke Kanies2010-02-172-1/+5
| | | | | | | | This disables adding any code to 'main' except in site.pp, so if you have code outside of a node, class, or define it will throw an exception. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing #1545 - Adding 'caller_module_name' variableLuke Kanies2010-02-172-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | This will produce the name of the module that a given resource is defined in, rather than the module that the resource type itself is defined in. For instance: # in one/manifests/onedef.pp define one::onedef { notice "Called $name from $caller_module_name" } # in two/manifests/init.pp class two { one::onedef { yay: } } produces: Called yay from two This could obviously be extended to actually build a caller stack, as frightening as that seems. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
* Fixing #1545 - module_name is now a variableLuke Kanies2010-02-173-5/+10
| | | | | | | | | | | | | This is only true for resource types (e.g., classes and defines) of course. The actual variable is 'module_name': class mymod { notify { "in mymod '$module_name'": } } Signed-off-by: Luke Kanies <luke@puppetlabs.com>