summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Merge branch 'maint/master/use-bracket-notation-in-specs'Nick Lewis2011-03-2914-13/+13
| |\ \ \
| | * | | maint: Use bracket notation instead of define in specsNick Lewis2011-03-2914-13/+13
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The intent of these specs is to find the String rather than to actually define it. Thus, the bracket notation is more semantically accurate than using Puppet::String#define. Reviewed-By: Pieter van de Bruggen
| * | | Merge branch 'tickets/master/6770'Pieter van de Bruggen2011-03-2922-197/+114
| |\ \ \
| | * | | (#6770) Fix Puppet::String#load_actions.Pieter van de Bruggen2011-03-291-1/+1
| | | | | | | | | | | | | | | | | | | | Reviewed-By: Nick Lewis
| | * | | (#6770) Change versioning; adopt :current over :latest.Pieter van de Bruggen2011-03-2822-197/+114
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per discussion with Luke, versions of an interface are first looked up by requiring 'puppet/interface/{name}', and secondarily looked up by requiring '{name}@{version}/puppet/interface/{name}' if the first failed. A version of `:current` can be used to represent the version living in 'puppet/interface/{name}'. Paired-With: Nick Lewis
| * | | Merge branch 'maint/master/puppet-strings-is-the-official-api-name'Daniel Pittman2011-03-2866-408/+408
| |\ \ \
| | * | | MAINT: the API is officially named "string" as of this moment.Daniel Pittman2011-03-2866-408/+408
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Now that we have settled on the final public name for the API, "Puppet::String", mass-rename and mass-edit all the files to follow. Reviewed-By: Randall Hansen <randall@puppetlabs.com>
| * | | MAINT: fix the misordered invocations in action.Daniel Pittman2011-03-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When initializing we need to set the name and interface before we do anything else, since the reasonable assumption for users is that those invariants are there when their setter is called. This allows someone to override the interface or name by misusing the call to new, but they could already screw up by passing the wrong values, so whatever.
| * | | Merge branch 'tickets/master/6770'Pieter van de Bruggen2011-03-2510-38/+247
| |\ \ \
| | * | | (#6770) Add support for version :latest.Pieter van de Bruggen2011-03-256-6/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Specifying a version of `:latest` will find the most recent version of the named interface installed in your RUBYLIB, and attempt to load that. This is unlikely to provide a stable dependency in the future, so should be used sparingly, acknowledging the dangers. Reviewed-By: Daniel Pittman
| | * | | (#6770) Add version lookup and comparison.Pieter van de Bruggen2011-03-252-0/+137
| | | | | | | | | | | | | | | | | | | | Reviewed-By: Jacob Helwig
| | * | | (#6770) Refactor Puppet::Interface#initialize.Pieter van de Bruggen2011-03-258-35/+30
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | P::I#initialize now takes a name and a version (and an optional block). The options hash has been removed, though it may be reintroduced if a legitimate use case can be made for it (so far, it's only been used for the version number). Reviewed-By: Jacob Helwig
| * | | Config#print action always returns nilLuke Kanies2011-03-252-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We were returning 'true', which was getting printed unnecessarily. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
| * | | Merge branch ↵Daniel Pittman2011-03-244-9/+49
| |\ \ \ | | | | | | | | | | | | | | | 'feature/master/6833-support-'script'-as-"short-form"-action-creation-mechanism'
| | * | | (#6833) support 'script' as a short form of 'action'Daniel Pittman2011-03-244-9/+49
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At the moment the action method is a fairly heavy tool: it provides a DSL, and is designed to allow substantial metadata to be added to the action. For some users this is low on value, since they just want to write a little script that drives things a bit differently. Which there is substantial value in the metadata, adding the capability to do these light-weight things quickly is valid. To meet this we add a script action; the contrast is: action :foo do # other metadata goes here invoke do |args| # method body goes here end end script :bar do |args| # method body goes here end # ...and if you want metadata, you have to add it in more ugly, procedural # ways, which we are not going to encourage. Reviewed-By: Pieter van de Bruggen <pieter@puppetlabs.com>
| * | | (Maint.) Require 'puppet/interface' in spec_helper.rbPieter van de Bruggen2011-03-231-0/+1
| | | | | | | | | | | | | | | | Paired-With: Nick Lewis
| * | | Merge branch 'tickets/master/6770'Pieter van de Bruggen2011-03-2347-155/+170
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: lib/puppet/interface/certificate.rb spec/unit/application/interface_base_spec.rb spec/unit/interface/interface_collection_spec.rb
| | * | | (#6770) Rename Puppet::Interface::interface method.Pieter van de Bruggen2011-03-2335-68/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Puppet::Interface::interface is now Puppet::Interface::define, also aliased to Puppet::Interface::[] for convenience. Paired-With: Nick Lewis
| | * | | (#6770) Changing versioning to semver.Pieter van de Bruggen2011-03-2344-93/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | More information about the versioning scheme can be found at http://semver.org. Paired-With: Nick Lewis
| | * | | (#6770) Add basic versioning for interfaces.Pieter van de Bruggen2011-03-2346-131/+148
| | | | | | | | | | | | | | | | | | | | Reviewed-By: Nick Lewis
| * | | | Propagating an argument to search out of core.Richard Crowley2011-03-231-4/+3
| | | | | | | | | | | | | | | | | | | | Puppet::SSL::Host.search (which will proxy to the new indirection) accepts an array of classes (now also class names) to limit the search. Currently, `puppet certificate list` limits itself to certificate requests to mimic the behavior of `puppet cert -l`.
| * | | | Added list action.Richard Crowley2011-03-232-12/+25
| | | | | | | | | | | | | | | | | | | | The common tasks of checking the --ca-location argument and becoming a CA process if necessary (that is, acting like a master) have been abstracted into the Application where they belong.
| * | | | Alter generate action to work on CSRs only.Richard Crowley2011-03-231-3/+4
| | | | |
| * | | | Adding Certficate#generateLuke Kanies2011-03-221-2/+12
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
| * | | | WIP - all tests failLuke Kanies2011-03-223-12/+20
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Luke Kanies <luke@puppetlabs.com>
| * | | | Use the new name for the terminus.Richard Crowley2011-03-232-8/+2
| | | | |
| * | | | Factoring cert status app back into certificate.Richard Crowley2011-03-233-0/+51
| | | | |
| * | | | Merge branch 'maint/master/fix-order-dependent-spec-failures'Nick Lewis2011-03-222-6/+11
| |\ \ \ \ | | |/ / / | |/| | |
| | * | | maint: Fix order-dependent spec failuresNick Lewis2011-03-222-6/+11
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The specs for InterfaceCollection were clearing the list of interfaces at the end of the spec run, which caused later specs to fail because they couldn't re-require interfaces they needed. This fixes the InterfaceCollection specs to save and restore the interfaces at the end of the file. Reviewed-By: Matt Robinson
| * | | Merge branch 'maint/master/always-use-interface-method'Nick Lewis2011-03-2234-340/+229
| |\ \ \
| | * | | maint: Implement an InterfaceCollection class to manage interfacesNick Lewis2011-03-2234-340/+229
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Having an instance variable on class Interface is insufficient for Interface::Indirector. This also changes the semantics of "Interface.interface" to handle registration and loading actions, and for "Interface.new" to only instantiate an Interface. Thus, consumers of the API should typically use "Interface.interface", unless they have reasons to not want an interface automatically registered. Paired-With: Pieter van de Bruggen
| * | | Merge branch 'ticket/master/6814'Nick Lewis2011-03-221-0/+1
| |\ \ \
| | * | | (#6814) Add missing require for specsNick Lewis2011-03-221-0/+1
| | | | |
| * | | | Merge branch 'maint/master/make-catalog-select-args-explicit'Nick Lewis2011-03-222-9/+9
| |\ \ \ \
| | * | | | maint: Make args to Catalog.select explicitNick Lewis2011-03-222-9/+9
| |/ / / / | | | | | | | | | | | | | | | No other args are used, so passing *args and shifting is unnecessary.
| * | | | Merge branch 'ticket/master/6814'Nick Lewis2011-03-2213-108/+323
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: lib/puppet/interface/catalog.rb lib/puppet/interface/catalog/select.rb lib/puppet/interface/configurer.rb
| | * | | (#6814) Create a dedicated Action classNick Lewis2011-03-2213-108/+323
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This class will represents an action, and allows us to store metadata for an action, and programmatically introspect and invoke them. A helper class ActionBuilder represents the DSL for defining an action. Also defined an "invoke" DSL method to handle the functionality of defining the method for an action. Reviewed-By: Daniel Pittman
| * | | | Merge branch 'tickets/master/6786'Pieter van de Bruggen2011-03-2220-82/+72
| |\ \ \ \ | | |/ / / | |/| | |
| | * | | (#6786) Change interface storage and access.Pieter van de Bruggen2011-03-2220-82/+72
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby's namespace mechanism introduced a number of problems, including incorrect name resolution for common and simple cases. Given that, we've refactored back to class-level data structures with accessor methods available. The current method names are unlikely to be the final UI. Reviewed-By: Daniel Pittman
| * | | Merge branch 'tickets/master/maint'Pieter van de Bruggen2011-03-213-28/+0
| |\ \ \
| | * | | (Maint.) Remove Puppet::Interface#unload_interfacePieter van de Bruggen2011-03-213-28/+0
| |/ / / | | | | | | | | | | | | Reviewed-By: Nick Lewis
| * | | Merge branch 'ticket/master/6805'Nick Lewis2011-03-218-0/+155
| |\ \ \
| | * | | (#6805) Add a "configurer" applicationNick Lewis2011-03-218-0/+155
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This application is similar in basic functionality to the "agent" application, but implemented in terms of interfaces. It currently will retrieve facts, retrieve a catalog, apply the catalog, and submit a report. Options such as noop and daemonize are still to come. Reviewed-By: Pieter van de Bruggen
| * | | Fixing a load-order issue in Puppet::InterfaceLuke Kanies2011-03-212-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The application classes were having issues loading the Interface class in certain circumstances because of load order. This just pushes the loading as late as possible. Signed-off-by: Luke Kanies <luke@puppetlabs.com>
| * | | Merge branch ↵Daniel Pittman2011-03-212-5/+32
| |\ \ \ | | | | | | | | | | | | | | | 'bug/master/6806-improve-error-checking-and-testing-of-interface-naming'
| | * | | (#6806) Improve error checking and reporting for interface naming.Daniel Pittman2011-03-212-5/+32
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | We didn't do enough input checking and sanitization, and missed some edge-cases for naming interfaces. This adds testing, and cleans up some edge cases to handle things better. Reviewed-By: Pieter van de Bruggen <pieter@puppetlabs.com>
| * | | Merge branch 'tickets/master/6785'Pieter van de Bruggen2011-03-211-3/+3
| |\ \ \
| | * | | (#6785) Internal consistency for `--terminus`.Pieter van de Bruggen2011-03-211-3/+3
| | | | | | | | | | | | | | | | | | | | Paired-With: Richard Crowley
| | * | | Merge branch 'master' into tickets/master/6785Pieter van de Bruggen2011-03-2114-29/+34
| | |\ \ \ | | |/ / / | |/| | |
| * | | | Merge branch 'tickets/master/6786'Pieter van de Bruggen2011-03-2114-29/+34
| |\ \ \ \