[openstack-dev] Proposed Logging Standards

Sean Dague sean at dague.net
Tue Jan 28 11:21:39 UTC 2014


On 01/27/2014 09:57 PM, Christopher Yeoh wrote:
> On Tue, Jan 28, 2014 at 12:55 AM, Sean Dague <sean at dague.net
> <mailto:sean at dague.net>> wrote:
> 
>     On 01/27/2014 09:07 AM, Macdonald-Wallace, Matthew wrote:
>     > Hi Sean,
>     >
>     > I'm currently working on moving away from the "built-in" logging
>     to use log_config=<filename> and the python logging framework so
>     that we can start shipping to logstash/sentry/<insert other useful
>     tool here>.
>     >
>     > I'd be very interested in getting involved in this, especially
>     from a "why do we have log messages that are split across multiple
>     lines" perspective!
> 
>     Do we have many that aren't either DEBUG or TRACE? I thought we were
>     pretty clean there.
> 
>     > Cheers,
>     >
>     > Matt
>     >
>     > P.S. FWIW, I'd also welcome details on what the "Audit" level
>     gives us that the others don't... :)
> 
>     Well as far as I can tell the AUDIT level was a prior drive by
>     contribution that's not being actively maintained. Honestly, I think we
>     should probably rip it out, because I don't see any in tree tooling to
>     use it, and it's horribly inconsistent.
> 
> 
> For the uses I've seen of it in the nova api code INFO would be
> perfectly fine in place of AUDIT.
> 
> I'd be happy to help out with patches to cleanup the logging in n-api.
> 
> One other thing to look at - I've noticed with logs is that when
> something like glanceclient code (just as an example) is called from nova,
> we can get ERROR level messages for say image not found when its
> actually perfectly expected that this will occur.
> I'm not sure if we should be changing the error level in glanceclient or
> just forcing any error logging in glanceclient when
> called from Nova to a lower level though.

It's now changed in glanceclient -
https://review.openstack.org/#/c/67744/ - it should be gone in the gate
logs, and will be gone for everyone once a new release is out.

	-Sean

-- 
Sean Dague
Samsung Research America
sean at dague.net / sean.dague at samsung.com
http://dague.net

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 482 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140128/34d48db3/attachment.pgp>


More information about the OpenStack-dev mailing list