On Sunday, November 20, 2011 at 11:25 AM, James Turnbull wrote:
> Trevor Vaughan wrote:
> > This was committed such that all file diffs would go into the logs.
> > 
> > Please be very vocal about announcing this and think about turning file 
> > diffs off as the default.
> > 
> > I think that most users are quite happy about being able to see diffs at 
> > the command line but not in the system logs since these may contain 
> > sensitive
> > information.
> > 
> > Also, you might want to have the --test command turn on diffs explicitly so 
> > that people don't have to. This would keep things working the way they are
> > now by default.
> > 
> 
> 
> Yeah I agree. The show_diff option should be off by default and added
> to --test.
> 
So this is the current behavior already. show_diff defaults to off, and we turn 
it on for --test and noop. Is the result here that it should not be turned on 
automatically for noop?
 
> 
> James
> 
> -- 
> James Turnbull
> Puppet Labs
> 1-503-734-8571
> To schedule a meeting with me: http://tungle.me/jamtur01
> 
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Puppet Developers" group.
> To post to this group, send email to puppet-dev@googlegroups.com 
> (mailto:puppet-dev@googlegroups.com).
> To unsubscribe from this group, send email to 
> puppet-dev+unsubscr...@googlegroups.com 
> (mailto:puppet-dev+unsubscr...@googlegroups.com).
> For more options, visit this group at 
> http://groups.google.com/group/puppet-dev?hl=en.
> 
> 


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

Reply via email to