Re: [Puppet-dev] sudoers type/provider

2010-03-25 Thread Dan Bode
Hi David, On Wed, Mar 24, 2010 at 1:29 AM, David Schmitt da...@dasz.at wrote: On 3/22/2010 9:04 PM, Dan Bode wrote: * Overloading the type with the three different types of records confuses me. Perhaps defines for each of the three types could improve the situation? This is

Re: [Puppet-dev] sudoers type/provider

2010-03-25 Thread David Schmitt
Hi Dan, Trevor, * On 3/25/2010 11:16 AM, Trevor Vaughan wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Inline... On 03/25/2010 04:09 AM, Dan Bode wrote: snip/ I can also imagine use cases where this would be too limiting. If two classes wanted to specify the same Cmnd_Alias for two

Re: [Puppet-dev] sudoers type/provider

2010-03-24 Thread David Schmitt
On 3/22/2010 9:04 PM, Dan Bode wrote: * Overloading the type with the three different types of records confuses me. Perhaps defines for each of the three types could improve the situation? This is not possible with parsedfile. Parsedfile is not able to synchronize reading/writing

Re: [Puppet-dev] sudoers type/provider

2010-03-22 Thread Dan Bode
On Wed, Mar 17, 2010 at 6:21 PM, Luke Kanies l...@reductivelabs.com wrote: On Mar 17, 2010, at 5:26 PM, Trevor Vaughan wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I agree with David. Also: - - I would make the defined attributes completely authoritative. If I'm setting up

Re: [Puppet-dev] sudoers type/provider

2010-03-22 Thread Dan Bode
On Wed, Mar 17, 2010 at 1:51 AM, David Schmitt da...@dasz.at wrote: On 3/17/2010 5:25 AM, Dan Bode wrote: Hi All, I have been working on a type/provider for sudoers and would appreciate any feedback. It can be found at: http://github.com/bodepd/puppet-sudo There are some slight

Re: [Puppet-dev] sudoers type/provider

2010-03-22 Thread Luke Kanies
On Mar 22, 2010, at 12:22 PM, Dan Bode wrote: I agree that this should be implemented. what hook can I use to implement functionality that occurs after everything has been flushed? Basically just change the default target to a tmp file or something, and modify flush to write the file,

Re: [Puppet-dev] sudoers type/provider

2010-03-22 Thread Dan Bode
On Mon, Mar 22, 2010 at 1:50 PM, Luke Kanies l...@reductivelabs.com wrote: On Mar 22, 2010, at 12:22 PM, Dan Bode wrote: I agree that this should be implemented. what hook can I use to implement functionality that occurs after everything has been flushed? Basically just change the default

Re: [Puppet-dev] sudoers type/provider

2010-03-17 Thread David Schmitt
On 3/17/2010 5:25 AM, Dan Bode wrote: Hi All, I have been working on a type/provider for sudoers and would appreciate any feedback. It can be found at: http://github.com/bodepd/puppet-sudo There are some slight limitations documented in the README. Plenty of examples in the tests directory

Re: [Puppet-dev] sudoers type/provider

2010-03-17 Thread Trevor Vaughan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I agree with David. Also: - - I would make the defined attributes completely authoritative. If I'm setting up sudoers, I want to define the entire file from top to bottom, I specifically don't want to have cruft left around. This could perhaps be an

Re: [Puppet-dev] sudoers type/provider

2010-03-17 Thread Luke Kanies
On Mar 17, 2010, at 1:51 AM, David Schmitt wrote: On 3/17/2010 5:25 AM, Dan Bode wrote: Hi All, I have been working on a type/provider for sudoers and would appreciate any feedback. It can be found at: http://github.com/bodepd/puppet-sudo There are some slight limitations documented in

Re: [Puppet-dev] sudoers type/provider

2010-03-17 Thread Luke Kanies
On Mar 17, 2010, at 5:26 PM, Trevor Vaughan wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I agree with David. Also: - - I would make the defined attributes completely authoritative. If I'm setting up sudoers, I want to define the entire file from top to bottom, I specifically

[Puppet-dev] sudoers type/provider

2010-03-16 Thread Dan Bode
Hi All, I have been working on a type/provider for sudoers and would appreciate any feedback. It can be found at: http://github.com/bodepd/puppet-sudo There are some slight limitations documented in the README. Plenty of examples in the tests directory to get people started. It's a pretty