Jujued logs

Ian Booth ian.booth at canonical.com
Thu Mar 28 22:34:02 UTC 2013


I agree, it's been annoying me too. The rationale behind it is that the format
of the log messages is:
<datetime> <severity> JUJU:<cmd> <msg text>

where, if command is a subcommand, it prints as <super cmd>:<cmd>

However, super command is always "juju" so that can easily be dropped from the
command name rendering.

As for the "cmd/juju" bit, that comes from the actual log message:

log.Infof("cmd/juju: looking blah blah...")

This would have been from before the log messages were formatted to include
standard fields and people had to add this sort of stuff ad hoc. So it's just a
matter of removing such text, and ensuring we do not add this sort of (now)
redundant information in the future to the message text itself.

On 29/03/13 06:41, Gustavo Niemeyer wrote:
> On Thu, Mar 28, 2013 at 5:20 PM, Gustavo Niemeyer
> <gustavo.niemeyer at canonical.com> wrote:
>> It feels like there's slightly too much jujuing on these lines:
>>
>>     2013/03/28 17:17:32 INFO JUJU:juju:publish cmd/juju:
>>
>> Hopefully it wouldn't be controversial to reduce that to more
>> reasonable jujuing levels? :-)
> 
> Hmmm.. a first step might be to just drop these cmd/juju prefixes
> within Infof logged messages under cmd/juju. That's pretty
> straightforward, and doesn't seem like a big deal (the log always
> comes from the executed command).
> 
> 
> gustavo @ http://niemeyer.net
> 



More information about the Juju-dev mailing list