[openstack-dev] [devstack][QA] Changing logging format to match documented defaults

Doug Hellmann doug at doughellmann.com
Tue Dec 22 23:44:15 UTC 2015


Excerpts from Ian Wienand's message of 2015-12-23 07:22:39 +1100:
> On 12/23/2015 05:55 AM, Ronald Bradford wrote:
> > I have observed that devstack uses custom logging formatting that
> > differs from the documented defaults. An example is for nova.  which
> > is defined in [1]
> 
> The original point mentioned there of using "*_name" for extra
> verbosity still seems relevant [1]
> 
> > logging_context_format_string = %(asctime)s.%(msecs)03d %(levelname)s
> > %(name)s [%(request_id)s *%(user_identity)s*] %(instance)s%(message)s
> 
> user_identity is still just "{user} {tenant} ..." (which is id's, I
> believe?) [2].
> 
> > This logging variable is also defined differently across 4
> > projects (nova, cinder, keystone and glance) so ultimately the goal would
> > be to ensure they may present documented configuration defaults.
> 
> Since it's hard enough to debug already when tempest gets going with
> multiple users/tenants, I think keeping the verbose logging is worth
> it.  However, if a bunch of copy-paste setting of this can be
> refactored, I'm sure we'd consider it favourably.

oslo.log 1.14.0 includes a new option to control the format of the
identity value, and that can be used to replace the ids with names. See
https://review.openstack.org/#/c/218139/5/oslo_log/_options.py

Doug

> 
> -i
> 
> [1] http://git.openstack.org/cgit/openstack-dev/devstack/commit/?id=6f13ba33d84b95808fc2a7672f332c1f0494e741
> [2] https://git.openstack.org/cgit/openstack/oslo.context/tree/oslo_context/context.py#n43
> 



More information about the OpenStack-dev mailing list