Issue #17319 has been updated by Jo Rhett.

I would argue that it is an implementation inconsistent with the existing 
functionality. If one were to read the descriptions of tagmail reports in 
conjunction with loglevel, the current implementation does not align with what 
they say.

At the very least, documentation for show_diff needs to be created which says 
"is not logged at the same level as the file resource" and "will show up in 
every report regardless of settings".  Because without those clarifications, it 
does not work as documented.

Anyway, this is all mincing words. I look forward to having this work as the 
documentation would suggest.
----------------------------------------
Feature #17319: show_diff should honor loglevel
https://projects.puppetlabs.com/issues/17319#change-75535

Author: Jo Rhett
Status: Needs Decision
Priority: Normal
Assignee: eric sorenson
Category: reports
Target version: 
Affected Puppet version: 2.7.19
Keywords: 
Branch: 


Right now the show_diff is shown every time the file is modified, no matter 
what loglevel the file changes are set for. This defeats the entire purpose of 
loglevel.

<pre>
On Tue, Mar 8, 2011 at 4:46 PM, Mohamed Lrhazi <[email protected]> wrote:
> I have resource for which I set this to debug, like so:
>
> file { "/tmp/foo":
>  content => inline_template("..."),
>  loglevel => debug,
> }
>
> I was expecting the following command to not log anything about it,
> but it does, it logs the diff of the file.. Why?

If you don't want to see the diff output when the file changes, turn
off the show_diff option.

Thanks,

Nan</pre>

Sure, Nan. Perhaps if we don't want to see every resource changed, we should 
turn off reporting altogether? Is this not exactly the purpose for loglevel?

Especially given that the diff is included with the resource report which is 
properly included or not by loglevel, this could be easily handled the same way.


-- 
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