On 2012-05-15 18:56, Jeff Weiss wrote:
Puppet Community,

We need your help.

We have a terrific contribution from Bart ten Brinke that would add an
`unless_uid` parameter to user resources.  The pull is request is 628
(https://github.com/puppetlabs/puppet/pull/628).

We acknowledge the clear use case around needing to exclude specific
users--not explicitly managed by Puppet--from a purge.  Because we don't
yet have an awesome resource collection syntax for this and we feel that
would be a decent-sized investment, we're on the fence about the syntax
that this would create.  Here's the example from the pull request:

|class users::resources {
   resources {'user':
   purge              =>  true,
   unless_system_user =>  true,
   unless_uid =>  [10_000..20_000];
   }
}|


To reiterate, we see value here and can't think of a better near-term
solution.

What are your thoughts? Is this intuitive enough? Is it a must-have feature?

Interesting concept. I find the syntax acceptable.

Alternatively a file-inspired syntax comes to mind:

    file { "/somewhere": source => "...", purge => true }

    user_range { "[10000..15000]": purge => false }

I'm not convinced that would be better.

A more concrete point would be to add more examples into the docstring. As it stands there is no chance to create a correct range expression from the documentation alone.


Related point: I'd really like to avoid *creating* users in a specified range. At a site I've got NIS users and puppet happily creates local admin users in that range creating clashes in the future. Any ideas?


Best Regards, David

--
You received this message because you are subscribed to the Google Groups "Puppet 
Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to