Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-12 Thread Michael DeHaan
Ok, I'll read up on Node.search ... I knew nothing about it before. I'll probably need some help here understanding what this design should look like. On Fri, Mar 12, 2010 at 1:56 AM, Luke Kanies l...@reductivelabs.com wrote: I'm pretty uncomfortable with this patch.  Does it replace the

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Michael DeHaan
On Thu, Mar 11, 2010 at 2:54 AM, James Turnbull ja...@lovedthanlost.net wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/03/10 1:51 AM, Michael DeHaan wrote: No, I haven't. I'm curious as to feedback on the idea/implementation. Tests can be added. Mike Actually I tend to

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Rein Henrichs
Michael, Things like glob parsing are both inherently testable and prone to errors. And, with respect, I don't know how to test this is not a valid argument for not testing. I'd be happy to talk to you about Puppet testing methodologies to the extent that they exist and that I am aware of them.

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Michael DeHaan
On Thu, Mar 11, 2010 at 1:56 PM, Rein Henrichs r...@reductivelabs.com wrote: Michael, Things like glob parsing are both inherently testable and prone to errors. Dir.glob is prone to errors? Personally, I think Puppet needs better testing methodologies just as much as it needs more and better 

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Luke Kanies
On Mar 11, 2010, at 6:17 AM, Michael DeHaan wrote: On Thu, Mar 11, 2010 at 2:54 AM, James Turnbull ja...@lovedthanlost.net wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/03/10 1:51 AM, Michael DeHaan wrote: No, I haven't. I'm curious as to feedback on the idea/implementation.

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Michael DeHaan
On Thu, Mar 11, 2010 at 2:50 PM, Luke Kanies l...@reductivelabs.com wrote: On Mar 11, 2010, at 6:17 AM, Michael DeHaan wrote: On Thu, Mar 11, 2010 at 2:54 AM, James Turnbull ja...@lovedthanlost.net wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/03/10 1:51 AM, Michael DeHaan

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Luke Kanies
On Mar 9, 2010, at 2:08 PM, Michael DeHaan wrote: Also cleaned up a few error messages (unrelated) --- lib/puppet/application/puppetrun.rb | 12 ++- lib/puppet/daemon.rb|2 +- lib/puppet/network/server.rb|2 +- lib/puppet/util/runner.rb | 53

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-11 Thread Luke Kanies
I'm pretty uncomfortable with this patch. Does it replace the direct ldap integration? If I used it with external nodes, what would happen? Do we expect our users to maintain these files in addition to their external nodes system? I'd much rather make sure that Node.search behaves like

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-10 Thread Michael DeHaan
No, I haven't. I'm curious as to feedback on the idea/implementation. Tests can be added. On Wed, Mar 10, 2010 at 1:32 AM, Paul Nasrat pnas...@googlemail.com wrote: On 9 March 2010 22:08, Michael DeHaan mich...@reductivelabs.com wrote: Also cleaned up a few error messages (unrelated) ---  

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-10 Thread James Turnbull
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/03/10 1:51 AM, Michael DeHaan wrote: No, I haven't. I'm curious as to feedback on the idea/implementation. Tests can be added. Mike Actually I tend to agree with Paul. We require tests for community submissions. Everyone should eat the

[Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-09 Thread Michael DeHaan
Also cleaned up a few error messages (unrelated) --- lib/puppet/application/puppetrun.rb | 12 ++- lib/puppet/daemon.rb|2 +- lib/puppet/network/server.rb|2 +- lib/puppet/util/runner.rb | 53 +++ 4 files changed,

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-09 Thread Todd Zullinger
Michael DeHaan wrote: +def expand_groups(host_list) +groupfile = #{Puppet[:confdir]}/hostgroups.conf Rather minor, but your commit subject says .txt but the code uses .conf. -- ToddOpenPGP - KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp

Re: [Puppet-dev] [PATCH] Add shell glob support to puppetrun, and also groups using /etc/puppet/hostgroups.txt (JSON format).

2010-03-09 Thread Paul Nasrat
On 9 March 2010 22:08, Michael DeHaan mich...@reductivelabs.com wrote: Also cleaned up a few error messages (unrelated) ---  lib/puppet/application/puppetrun.rb |   12 ++-  lib/puppet/daemon.rb                |    2 +-  lib/puppet/network/server.rb        |    2 +-