Issue #9160 has been updated by Stefan Schulte.

Status changed from In Topic Branch Pending Review to Tests Insufficient

Hi Digant,

your patch looks good but it breaks a test in `test/ral/manager/attributes.rb` 
(which can be easily resolved). Just in case I missed it: Is there a patch on 
puppet-dev or a branch on some git repository I can pull your work from?
----------------------------------------
Bug #9160: type.rb logs ridiculous information to info
https://projects.puppetlabs.com/issues/9160

Author: Digant Kasundra
Status: Tests Insufficient
Priority: Normal
Assignee: 
Category: logging
Target version: 2.7.x
Affected Puppet version: 2.7.1
Keywords: 
Branch: 


On each puppet run, there are 5 info level logs lines PER USER for each of the 
useradd features that are not supported because we aren't running on Solaris or 
AIX.  This should be a debug level log.  The following patch should fix.

    --- type.rb.orig        2011-08-22 17:35:53.000000000 -0700
    +++ type.rb     2011-08-22 17:35:00.000000000 -0700
    @@ -496,7 +496,7 @@
    if provider and ! provider.class.supports_parameter?(klass)
    missing = klass.required_features.find_all { |f| ! 
provider.class.feature?(f) }
    -      info "Provider %s does not support features %s; not managing 
attribute %s" % [provider.class.name, missing.join(", "), name]
    +      debug "Provider %s does not support features %s; not managing 
attribute %s" % [provider.class.name, missing.join(", "), name]
    return nil
    end


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en.

Reply via email to