[Puppet-dev] [PATCH/facter 1/1] (#6728) Facter improperly detects openvzve on CloudLinux systems

2011-04-06 Thread Ben Hughes
Make the openvz check for more than just the vz directory to be sure it's OpenVZ. Update the OpenVZ fact to be slightly more resilient in it's checking of OpenVZ, so it doesn't clash with CloudLinux's LVE container system. Signed-off-by: Ben Hughes --- lib/facter/util/virtual.rb |2 +-

[Puppet-dev] [PATCH/puppet 1/2] (#6490) Add plugin initialization callback system to core

2011-04-06 Thread Markus Roberts
The recurring pattern "drop-in code needs to have something done at startup" is presently being solved with a variety of ad-hock mechanism. This commit adds a general, extensible, centralized system for adding such hooks and manages an extensible set of metadata about plugins which it collects by

[Puppet-dev] [PATCH/puppet 2/2] (maint) Indentation fixes

2011-04-06 Thread Markus Roberts
Signed-off-by: Markus Roberts --- lib/puppet/application.rb |8 lib/puppet/util/command_line.rb | 20 +--- 2 files changed, 13 insertions(+), 15 deletions(-) diff --git a/lib/puppet/application.rb b/lib/puppet/application.rb index 5e69bae..a028a15 100644 ---

[Puppet-dev] ANNOUNCE: Facter 1.5.9rc5

2011-04-06 Thread Jacob Helwig
Facter 1.5.9rc5 is a maintenance release containing fixes and updates. This release candidate addresses an issue with EC2 facts. This ensures that ARP facts are only returned for EC2 hosts. (#6976) As always, please let us know if you run into any problems with any of the release candidates. Th

[Puppet-dev] [PATCH/puppet 8/8] (5200) -- replace containers with sentinals

2011-04-06 Thread Markus Roberts
This commit removes the last remaining use of topsort (in SimpleGraph#splice!) by fixing #5200 in a way that is compatible with graph fontiers. Instead of replacing containers with many-to-many relationships, we now replace them with a pair of sentinals (whits) that bracket them. Thus a graph c

[Puppet-dev] [PATCH/puppet 6/8] (6911) Core change -- replace topsort with frontier ordered by salted SHA1

2011-04-06 Thread Markus Roberts
This is the core change of the ticket; rather than using a topological sort to statically determine the order in which resources should be applied, we use the graph wrapper introduced in the prior commit to dynamically determine the order in which to apply resources based on 1) the status of the

[Puppet-dev] [PATCH/puppet 7/8] (6911) Cleanup and renaming of transaction internals

2011-04-06 Thread Markus Roberts
The preceeding changes left some rough edges in the Transactions (a short, badly named method that was only used in one place and would be clearer in- line, a return value that was carfully retained and never used, etc.) This commit clears some of that up. Signed-off-by: Markus Roberts --- Local-

[Puppet-dev] [PATCH/puppet 5/8] (6911) Add bookkeeping facade around Transaction#relationship_graph

2011-04-06 Thread Markus Roberts
To implement graph frontiers transactions need to track information about the catalog's relationship graph. For various reasons (serialzation, lifetime, etc.) the data belongs with the transaction rather than the catalog or its relationship graph. This commit introduces a facade around the proper

[Puppet-dev] [PATCH/puppet 4/8] (6911) Cleanup of generate_additional_resources

2011-04-06 Thread Markus Roberts
The previous commit left only one meaningful value for the method parameter of generate_additional_resources, making it a constant not a parameter. This commit removes it. Paired-with: Jesse Wolfe Signed-off-by: Markus Roberts --- Local-branch: feature/next/resource_application_order lib/puppe

[Puppet-dev] [PATCH/puppet 3/8] (6911) Refactor to localize eval_generate dependency assumptions

2011-04-06 Thread Markus Roberts
To implement #6911 we will need to be able to make incremental descisions about order of application based only on the contents of the resource graph and local "working data." This commit begins to pull the needed structure into a method (visit_resources) while, for the moment, maintaining the ori

[Puppet-dev] [PATCH/puppet 1/8] (6911) Use normal methods to implement "depthfirst?" test

2011-04-06 Thread Markus Roberts
There was a class instance variable that was used to determine if a resource types's children should be processed before or after the parent, to support the one type (tidy) which did this. Instead, we define a normal function in Type to return false and override it in Tidy to return true. Paired-

[Puppet-dev] feature resource_application_order against next

2011-04-06 Thread Markus Roberts
This is a rather complex set of structural changes. But to understand it you really only need to grasp two key sections; the rest is cleanup. Cleanup commits: The thrust of these is to localize the code that we are going to change (the itterator over the dependency graph) and get all th

[Puppet-dev] [PATCH/puppet 2/8] (maint) Fix for require order issue

2011-04-06 Thread Markus Roberts
The recent AIX work added a dependency on Puppet::Parameter::Keyvalue in the group type, but didn't add the requisite require, causing failures under some load orders. Signed-off-by: Markus Roberts --- Local-branch: feature/next/resource_application_order lib/puppet/type/group.rb |1 + 1 fil

Re: [Puppet-dev] [PATCH/puppet 1/1] Fixed #5684 - Move to Apache 2.0 license

2011-04-06 Thread James Turnbull
Todd Zullinger wrote: > James Turnbull wrote: >> --- a/conf/redhat/puppet.spec >> +++ b/conf/redhat/puppet.spec >> @@ -8,7 +8,7 @@ Name: puppet >> Version:2.6.0 >> Release:1%{?dist} >> Summary:A network tool for managing many disparate systems >> -License:

[Puppet-dev] [PATCH/puppet-dashboard 1/1] (#6988) rpm should include logrotate file

2011-04-06 Thread Andreas Zuber
logrotate snippet added to rpm package config folder Signed-off-by: Andreas Zuber --- Local-branch: ticket/master/6988 ext/packaging/redhat/puppet-dashboard.logrotate | 10 ++ ext/packaging/redhat/puppet-dashboard.spec |1 + 2 files changed, 11 insertions(+), 0 deletions(-) c