openstack
/
ironic
Code Issues Proposed changes

Fix oslo.messaging log level 

The default log level of oslo.messaging was set on the python package
name, which means it didn't take effect for all logs. This fixes it
adding it on the namespace name as well.

Related-Bug: #1685148
Change-Id: I0263717b9fd4d3c6ee4083c71df9b4b26401a39b
This commit is contained in:
Thomas Herve 2017-04-19 10:11:49 +02:00
parent 350d8dec39
commit 674ddb0268
3 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
etc/ironic/ironic.conf.sample
View File

@ -463,7 +463,7 @@
# List of package logging levels in logger=LEVEL pairs. This
# option is ignored if log_config_append is set. (list value)
#default_log_levels = amqp=WARNING,amqplib=WARNING,qpid.messaging=INFO,oslo_messaging=INFO,sqlalchemy=WARNING,stevedore=INFO,eventlet.wsgi.server=INFO,iso8601=WARNING,paramiko=WARNING,requests=WARNING,neutronclient=WARNING,glanceclient=WARNING,urllib3.connectionpool=WARNING,keystonemiddleware.auth_token=INFO,keystoneauth.session=INFO
#default_log_levels = amqp=WARNING,amqplib=WARNING,qpid.messaging=INFO,oslo_messaging=INFO,oslo.messaging=INFO,sqlalchemy=WARNING,stevedore=INFO,eventlet.wsgi.server=INFO,iso8601=WARNING,paramiko=WARNING,requests=WARNING,neutronclient=WARNING,glanceclient=WARNING,urllib3.connectionpool=WARNING,keystonemiddleware.auth_token=INFO,keystoneauth.session=INFO
# Enables or disables publication of error events. (boolean
# value)
@ -2564,10 +2564,20 @@
# value)
#sasl_config_name =
# User name for message broker authentication (string value)
# DEPRECATED: User name for message broker authentication
# (string value)
# This option is deprecated for removal.
# Its value may be silently ignored in the future.
# Reason: Should use configuration option transport_url to
# provide the username.
#username =
# Password for message broker authentication (string value)
# DEPRECATED: Password for message broker authentication
# (string value)
# This option is deprecated for removal.
# Its value may be silently ignored in the future.
# Reason: Should use configuration option transport_url to
# provide the password.
#password =
# Seconds to pause before attempting to re-connect. (integer

3
ironic/conf/opts.py
View File

@ -88,7 +88,10 @@ def update_opt_defaults():
'amqp=WARNING',
'amqplib=WARNING',
'qpid.messaging=INFO',
# TODO(therve): when bug #1685148 is fixed in oslo.messaging, we
# should be able to remove one of those 2 lines.
'oslo_messaging=INFO',
'oslo.messaging=INFO',
'sqlalchemy=WARNING',
'stevedore=INFO',
'eventlet.wsgi.server=INFO',

4
releasenotes/notes/messaging-log-level-5f870ea69db53d26.yaml Normal file
View File

@ -0,0 +1,4 @@
---
fixes:
- Fixes an issue where oslo.messaging DEBUG logs were being logged when, by
default, they shouldn't have been.