[Puppet-dev] [PATCH/facter 1/1] Fixed #3939 - Rescue action for swap -l on AIX

2010-06-10 Thread James Turnbull
Signed-off-by: James Turnbull ja...@lovedthanlost.net --- lib/facter/memory.rb | 14 +- 1 files changed, 9 insertions(+), 5 deletions(-) diff --git a/lib/facter/memory.rb b/lib/facter/memory.rb index f0f0d45..94769d0 100644 --- a/lib/facter/memory.rb +++ b/lib/facter/memory.rb @@

Re: [Puppet-dev] 3962 - 0.25.5 fails to start if /var/lib does not exist

2010-06-10 Thread David Schmitt
On 6/9/2010 6:19 PM, Markus Roberts wrote: I'd advocate a mkdir -p style solution, unless anyone can see problems with that. Well, I would say that we should either be localizing or not. We're now defaulting to a value that works great for the linux world but doesn't even work in the

Re: [Puppet-dev] Composite primary keys as namevars

2010-06-10 Thread David Schmitt
On 6/9/2010 3:08 PM, Luke Kanies wrote: On Jun 9, 2010, at 3:27 AM, David Schmitt wrote: On 6/8/2010 7:18 PM, Markus Roberts wrote: David -- I'll leave the final decision to you. Just let me restate that I believe that having the additional render or to_s block (what I called

Re: [Puppet-dev] Re: Query facts of other nodes, or something like that

2010-06-10 Thread David Schmitt
On 6/10/2010 3:36 AM, Daniel Pittman wrote: Luke Kaniesl...@puppetlabs.com writes: 1) Easy way to force updates when a required catalog has been changed. (This doesn't automatically fix the problem of new data showing up, only changed data.) 2) With good graph viewing, a

Re: [Puppet-dev] Re: Query facts of other nodes, or something like that

2010-06-10 Thread R.I.Pienaar
- David Schmitt da...@dasz.at wrote: (OTOH, I also think that the whole storeconfigs thing is probably a proxy for an mcollective live query type operation across the infrastructure, so my ideas may be completely crazy :) There is an overlap between the two, but both provide

Re: [Puppet-dev] [PATCH/facter 1/1] Fixed #3939 - Rescue action for swap -l on AIX

2010-06-10 Thread Paul Nasrat
On 10 June 2010 07:16, James Turnbull ja...@lovedthanlost.net wrote: Signed-off-by: James Turnbull ja...@lovedthanlost.net -1 Hmm, there are a whole bunch of facts/commands that require root or behave differently. I'd rather see this implemented as a confine. ---  lib/facter/memory.rb |  

Re: [Puppet-dev] [PATCH/facter 1/1] Fixed #3939 - Rescue action for swap -l on AIX

2010-06-10 Thread Héctor Rivas Gándara
I think it simply could detect that the command (swap -l ) failed and raise an exception with the output (or first line of the output). Now it fails with a nil access error, I had to see the code to determine that the problem was that I was running facter as a normal user and 'swap -l' was

Re: [Puppet-dev] [PATCH/facter 1/1] Fixed #3939 - Rescue action for swap -l on AIX

2010-06-10 Thread Paul Nasrat
2010/6/10 Héctor Rivas Gándara key...@gmail.com: I think it simply could detect that the command (swap -l ) failed and raise an exception with the output (or first line of the output). Potentially yes, we should change how util.exec works and enable logging to user/exception passing to lib

[Puppet-dev] [PATCH/facter 1/1] Fixed #3929 - Added user confine to AIX memory facts

2010-06-10 Thread James Turnbull
Signed-off-by: James Turnbull ja...@lovedthanlost.net --- lib/facter/memory.rb | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/facter/memory.rb b/lib/facter/memory.rb index f0f0d45..06640e6 100644 --- a/lib/facter/memory.rb +++ b/lib/facter/memory.rb @@

Re: [Puppet-dev] [PATCH/puppet 1/1] Fixing #3988 - adding support for watchr

2010-06-10 Thread David Schmitt
On 10.06.2010 18:55, Luke Kanies wrote: Signed-off-by: Luke Kaniesl...@puppetlabs.com --- autotest/images/fail.png| Bin 0 - 19978 bytes autotest/images/pass.png| Bin 0 - 21763 bytes autotest/images/pending.png | Bin 0 - 19745 bytes autotest/watcher.rb | 124

Re: [Puppet-dev] [PATCH/puppet 1/1] Fixing #3988 - adding support for watchr

2010-06-10 Thread Luke Kanies
On Jun 10, 2010, at 2:01 PM, David Schmitt wrote: On 10.06.2010 18:55, Luke Kanies wrote: Signed-off-by: Luke Kaniesl...@puppetlabs.com --- autotest/images/fail.png| Bin 0 - 19978 bytes autotest/images/pass.png| Bin 0 - 21763 bytes autotest/images/pending.png | Bin 0 - 19745

Re: [Puppet-dev] When to fail? (#3933 option to make Puppet fail as soon as a resource fails.)

2010-06-10 Thread Luke Kanies
On Jun 9, 2010, at 10:46 PM, Nigel Kersten wrote: So this was reasonably obvious on 0.25.x, but the master changes I'm less sure about... Basically the idea is that for debugging purposes we have a flag like --exitonfail and we exit as soon as a resource fails to apply. The whole transaction

Re: [Puppet-dev] Re: Query facts of other nodes, or something like that

2010-06-10 Thread Luke Kanies
On Jun 9, 2010, at 9:31 PM, Daniel Pittman wrote: Luke Kanies l...@puppetlabs.com writes: On Jun 7, 2010, at 7:42 PM, Daniel Pittman wrote: Thank you all for taking this seriously, especially when all I have are requests, rather than code to back them up. I really appreciate it, and

Re: [Puppet-dev] Re: Query facts of other nodes, or something like that

2010-06-10 Thread Luke Kanies
On Jun 10, 2010, at 3:36 AM, R.I.Pienaar wrote: - David Schmitt da...@dasz.at wrote: (OTOH, I also think that the whole storeconfigs thing is probably a proxy for an mcollective live query type operation across the infrastructure, so my ideas may be completely crazy :) There is