Issue #12051 has been updated by Daniel Pittman.

Status changed from In Topic Branch Pending Review to Merged - Pending Release
Target version changed from 2.7.x to 2.7.11

Thanks - that is awesome, and totally a sane thing to do.  We should have been 
doing this forever. :)
----------------------------------------
Bug #12051: Exec onlyif / unless does not capture output
https://projects.puppetlabs.com/issues/12051

Author: Robert Buchholz
Status: Merged - Pending Release
Priority: Normal
Assignee: 
Category: exec
Target version: 2.7.11
Affected Puppet version: 2.6.4
Keywords: exec logging
Branch: https://github.com/puppetlabs/puppet/pull/362


Puppet does not log the output of a check command to an exec, even if logoutput 
is true. There are no other facilities to log the check output either.

Example:
    exec { "/bin/something":
      onlyif => "/bin/check",
      logoutput => true,
    }

Results in    

    debug: /Stage[main]/redacted/Exec[/bin/something]: Executing check 
'/bin/check'                                                                    
                                                                             
    debug: Executing '/bin/check'
    debug: ... next entity

Workaround: I can append " > /path/to/logfile 2>&1" to the check command. But 
that's what I would expect "logoutput" to be there for. The way it is right now 
makes it unnecessarily hard to debug scripts that fail inside puppet but not 
outside (due to environment differences, for example).



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