Re: [Puppet-dev] Re: ANNOUNCE: Puppet 2.6.5 released!

2011-02-28 Thread Thomas S Hatch
On Mon, Feb 28, 2011 at 6:54 PM, Nigel Kersten wrote: > Sorry, forgot the release notes: > > https://projects.puppetlabs.com/projects/puppet/wiki/Release_Notes > > Thanks Puppet Labs for yet another great release! and Todd beats me to the punch again on the fedora packages :) Arch Linux packag

Re: [Puppet-dev] ANNOUNCE: Puppet 2.6.5 released!

2011-02-28 Thread Todd Zullinger
Nigel Kersten wrote: > The release is available for download at: > > http://puppetlabs.com/downloads/puppet/puppet-2.6.5.tar.gz Congratulations and thanks to all the folks who worked on the release! For those using Fedora or RHEL/CentOS, I've updated the yum repos at: http://tmz.fedorapeople

[Puppet-dev] Re: ANNOUNCE: Puppet 2.6.5 released!

2011-02-28 Thread Nigel Kersten
Sorry, forgot the release notes: https://projects.puppetlabs.com/projects/puppet/wiki/Release_Notes -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to puppet-dev@googlegroups.com. To unsubscribe from this

[Puppet-dev] ANNOUNCE: Puppet 2.6.5 released!

2011-02-28 Thread Nigel Kersten
The release is available for download at: http://puppetlabs.com/downloads/puppet/puppet-2.6.5.tar.gz Please note that this release of Puppet is signed with the Puppet Labs key. See the Verifying Puppet Download section at: http://projects.puppetlabs.com/projects/puppet/wiki/Downloading_Puppet R

[Puppet-dev] [PATCH/puppet 2/3] (#5466) Monkey patch Symbol so that you can sort them

2011-02-28 Thread Matt Robinson
It turns out that the ability to sort symbols comes in the preinit section of application run when we load Facter and hit the code that adds the <=> method for symbols in lib/facter/util/plist/generator.rb Reviewed-by: Nick Lewis Signed-off-by: Matt Robinson --- lib/puppet/util/monkey_patches.r

[Puppet-dev] [PATCH/puppet 1/3] (#5466) Fixed puppet resource bug with trailing ,

2011-02-28 Thread Matt Robinson
From: Nan Liu Signed-off-by: Matt Robinson --- lib/puppet/resource.rb | 29 - 1 files changed, 20 insertions(+), 9 deletions(-) diff --git a/lib/puppet/resource.rb b/lib/puppet/resource.rb index e832804..a71675e 100644 --- a/lib/puppet/resource.rb +++ b/lib/puppe

[Puppet-dev] [PATCH/puppet 3/3] (#5466) Write specs for output of puppet resource

2011-02-28 Thread Matt Robinson
Reviewed-by: Nick Lewis Signed-off-by: Matt Robinson --- spec/unit/resource_spec.rb | 34 +++--- 1 files changed, 19 insertions(+), 15 deletions(-) diff --git a/spec/unit/resource_spec.rb b/spec/unit/resource_spec.rb index ff31b24..eaa3d55 100755 --- a/spec/unit/re

Re: [Puppet-dev] [PATCH 1/1] (#5466) Fixed puppet resource bug with trailing ,

2011-02-28 Thread Matt Robinson
This causes failing tests. I thought it would be really easy to fix the tests, but it turns out that the ability to sort symbols comes in the preinit section of application run when we load Facter and hit the code that adds the <=> method for symbols in lib/facter/util/plist/generator.rb I'm goin

Re: [Puppet-dev] [PATCH/puppet 1/1] (#4922) Don't truncate remotely-sourced files on 404

2011-02-28 Thread Daniel Pittman
On Mon, Feb 28, 2011 at 14:45, Nick Lewis wrote: > We were 'handling' 404's on remote file content retrieval by returning nil > rather than raising an exception. This caused no content to be written to the > temporary file, but still appeared successful, so the destination file was > overwritten,

[Puppet-dev] [PATCH/puppet 1/1] (#4922) Don't truncate remotely-sourced files on 404

2011-02-28 Thread Nick Lewis
We were 'handling' 404's on remote file content retrieval by returning nil rather than raising an exception. This caused no content to be written to the temporary file, but still appeared successful, so the destination file was overwritten, instead of preserved. Now we just handle 404 like any oth

[Puppet-dev] [PATCH/puppet 7/8] (#6509) Inline docs: Fix erroneous code block in directoryservice provider for computer type

2011-02-28 Thread nfagerlund
Signed-off-by: nfagerlund --- lib/puppet/provider/computer/computer.rb |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/computer/computer.rb b/lib/puppet/provider/computer/computer.rb index a6be6bd..dd055be 100644 --- a/lib/puppet/provider/computer

[Puppet-dev] [PATCH/puppet 8/8] Maint: Align tabs in a code block in the Augeas type.

2011-02-28 Thread nfagerlund
Signed-off-by: nfagerlund --- lib/puppet/type/augeas.rb |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/puppet/type/augeas.rb b/lib/puppet/type/augeas.rb index d29bda6..a8fb1f1 100644 --- a/lib/puppet/type/augeas.rb +++ b/lib/puppet/type/augeas.rb @@ -98,10 +9

[Puppet-dev] [PATCH/puppet 6/8] Maint: Rewrite comments about symlinks to reflect best practice.

2011-02-28 Thread nfagerlund
Don't use ensure => 'path/to/file', because it's hard to read. Use ensure => link and specify a target =>. Signed-off-by: nfagerlund --- lib/puppet/type/file/ensure.rb | 28 +++- lib/puppet/type/file/target.rb | 15 ++- 2 files changed, 25 insertions(+), 1

[Puppet-dev] [PATCH/puppet 5/8] (#6509) Inline docs: Fix broken lists in Launchd provider.

2011-02-28 Thread nfagerlund
Lists need a leading linebreak. Signed-off-by: nfagerlund --- lib/puppet/provider/service/launchd.rb | 19 +++ 1 files changed, 11 insertions(+), 8 deletions(-) diff --git a/lib/puppet/provider/service/launchd.rb b/lib/puppet/provider/service/launchd.rb index 1632eda..07c549a

[Puppet-dev] [PATCH/puppet 4/8] (#6509) Inline docs: Fix broken code blocks in zpool type

2011-02-28 Thread nfagerlund
This one was subtle. If the first paragraph of a Markdown string embedded in a type isn't multiple lines, a code block immediately following it will not be recognized. So, hard-wrap or die, I guess. Signed-off-by: nfagerlund --- lib/puppet/type/zpool.rb | 12 +++- 1 files changed, 7

[Puppet-dev] [PATCH/puppet 3/8] (#6509) Inline docs: Fix code blocks in service type.

2011-02-28 Thread nfagerlund
Signed-off-by: nfagerlund --- lib/puppet/provider/service/daemontools.rb | 12 ++-- lib/puppet/provider/service/runit.rb |8 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/lib/puppet/provider/service/daemontools.rb b/lib/puppet/provider/service/dae

[Puppet-dev] [PATCH/puppet 1/8] (#6509) Inline docs: Fix broken code block in file type (content attribute)

2011-02-28 Thread nfagerlund
Signed-off-by: nfagerlund --- lib/puppet/type/file/content.rb | 24 1 files changed, 12 insertions(+), 12 deletions(-) diff --git a/lib/puppet/type/file/content.rb b/lib/puppet/type/file/content.rb index 63c0aaf..0e31f30 100755 --- a/lib/puppet/type/file/content.rb ++

[Puppet-dev] [PATCH/puppet 2/8] (#6509) Inline docs: fix broken code blocks in schedule.rb.

2011-02-28 Thread nfagerlund
Signed-off-by: nfagerlund --- lib/puppet/type/schedule.rb | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/lib/puppet/type/schedule.rb b/lib/puppet/type/schedule.rb index 82f17e5..5fb008f 100755 --- a/lib/puppet/type/schedule.rb ++

[Puppet-dev] (#6509) Fix broken code blocks scattered throughout types and providers

2011-02-28 Thread nfagerlund
This patch set fixes a collection of Markdown glitches in the inline type and provider docs. All of these errors affect http://docs.puppetlabs.com/references/[version]/type.html. -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to

Re: [Puppet-dev] Getting errors when sending to the list

2011-02-28 Thread James Turnbull
Stefan Schulte wrote: > For each message I'm sending to puppet-dev or puppet-users I get a mail > in my inbox (since 26th Feb): > > > The message that you sent was undeliverable to the following: > > Mdelves (User account is expired) > > > The sender is mailer-dae...@mthelen7.ballarat.

[Puppet-dev] Getting errors when sending to the list

2011-02-28 Thread Stefan Schulte
For each message I'm sending to puppet-dev or puppet-users I get a mail in my inbox (since 26th Feb): The message that you sent was undeliverable to the following: Mdelves (User account is expired) The sender is mailer-dae...@mthelen7.ballarat.edu.au. Do also have this problem? -Stef

Re: [Puppet-dev] RFC: Database-backed inventory service plan

2011-02-28 Thread Markus Roberts
I am not a DBA, but I've seen the following pattern play out many times: the DBA says "normalize, normalize!" the developers say "normalizing is a pain, and in this particular case it doesn't buy us much, etc." and down the road the DBA turns out to have been right. Then the developers rediscover

Re: [Puppet-dev] RFC: Database-backed inventory service plan

2011-02-28 Thread Matt Robinson
I'm less concerned with the performance ramifications or specifics of schema design of my proposal than the fact that I don't want to tie the architecture of the inventory service to storeconfigs, but performace concerns are addressed inline farther down this email. On Mon, Feb 28, 2011 at 5:43 AM

Re: [Puppet-dev] RFC: Database-backed inventory service plan

2011-02-28 Thread Ohad Levy
On Fri, Feb 25, 2011 at 4:55 PM, Matt Robinson wrote: > On Wed, Feb 23, 2011 at 2:04 PM, Nick Lewis wrote: > > Our current plan for the inventory service is to provide active_record > > termini for the "facts" and "inventory" indirections. This is to support > > fast look-up of facts, and search

Re: [Puppet-dev] [PATCH/puppet 1/8] Instrumentation foundation layer

2011-02-28 Thread Brice Figureau
On Sun, 2011-02-27 at 14:07 -0800, Markus Roberts wrote: > B - > > > + def self.listeners_of(label) > > +synchronize { > > + @listeners_of[label] ||= @listeners.select do | > k,l| > > +l.listen_to?(label) > > +