Re: [Puppet-dev] [PATCH/puppet 1/1] (#7137) Get rid of spurious info messages in useradd

2011-04-26 Thread Nigel Kersten
On Mon, Apr 25, 2011 at 11:41 PM, Matt Robinson wrote: > If you read the ticket associated with this patch you'll see the original > author of the AIX code (Hector Rivas) suggested removing the default, so I'm > not terribly worried about AIX. > > I agree it is possible that there is an issue wit

Re: [Puppet-dev] [PATCH/puppet 1/1] (#7137) Get rid of spurious info messages in useradd

2011-04-25 Thread Luke Kanies
Thank you - this was quite annoying. -- http://puppetlabs.com/ | +1-615-594-8199 | @puppetmasterd On Apr 25, 2011, at 10:00 PM, Max Martin wrote: > Usage of the useradd provider was leading to spurious log messages of > this form: > > info: /User[nigel]: Provider useradd does not support featu

Re: [Puppet-dev] [PATCH/puppet 1/1] (#7137) Get rid of spurious info messages in useradd

2011-04-25 Thread Matt Robinson
If you read the ticket associated with this patch you'll see the original author of the AIX code (Hector Rivas) suggested removing the default, so I'm not terribly worried about AIX. I agree it is possible that there is an issue with defaults in providers that should be addressed, but it's interes

Re: [Puppet-dev] [PATCH/puppet 1/1] (#7137) Get rid of spurious info messages in useradd

2011-04-25 Thread Daniel Pittman
So, this fixes the problem on non-AIX platforms. What changes now on AIX where we *don't* have a default value for that parameter? I worry that we are working around a known bug (inapplicable parameters with defaults cause warnings) by changing behaviour on AIX, which I understand poorly enough t

[Puppet-dev] [PATCH/puppet 1/1] (#7137) Get rid of spurious info messages in useradd

2011-04-25 Thread Max Martin
Usage of the useradd provider was leading to spurious log messages of this form: info: /User[nigel]: Provider useradd does not support features manages_aix_lam; not managing attribute ia_load_module This was due to the ia_load_module parameter requiring manages_aix_lam and additionally having a d