search

phinze / puppet-sudoers

puppet module for managing sudoers
MIT License
8 stars 9 forks source link

Option to require or not the User/Groups #5

Closed etlweather closed 9 years ago

etlweather commented 9 years ago

I'm working with PBIS and use Active Directory groups for the sudoer files so the group does not exist in Puppet. This option makes it that if set to false (default is true), it won't set a require parameter.

Are you planning to push this to the Puppet forge? Your built module there is pretty old.

phinze commented 9 years ago

Hey thanks for this! Reviewing now...

phinze commented 9 years ago

Looks good to me - I was trying to slap a test around it and fighting with the latest version of rspec-puppet, but I'll just do that on master.

Are you planning to push this to the Puppet forge? Your built module there is pretty old.

Sure! Once I get the build sorted I'll tag a release and push there.

Thanks again! :cake:

phinze commented 9 years ago

https://forge.puppetlabs.com/phinze/sudoers <-- 0.1.1 released :+1:

etlweather commented 9 years ago

Great!