From 8beceb2d6b5f7f584379c77edada1a9597834c71 Mon Sep 17 00:00:00 2001 From: Dmitry Tantsur Date: Mon, 3 Aug 2015 17:34:22 +0200 Subject: [PATCH] Log configuration values as DEBUG, not INFO According to the OpenStack logging standards, INFO logging should contain unit of works and should be targetted at operators: https://wiki.openstack.org/wiki/LoggingStandards#INFO Change-Id: Id584905e9b11136e674066d676c533d0c503fd61 --- ironic/cmd/api.py | 4 ++-- ironic/cmd/conductor.py | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/ironic/cmd/api.py b/ironic/cmd/api.py index b0b8a80c01..58ae77fb09 100644 --- a/ironic/cmd/api.py +++ b/ironic/cmd/api.py @@ -53,8 +53,8 @@ def main(): LOG = log.getLogger(__name__) LOG.info(_LI("Serving on http://%(host)s:%(port)s"), {'host': host, 'port': port}) - LOG.info(_LI("Configuration:")) - CONF.log_opt_values(LOG, logging.INFO) + LOG.debug("Configuration:") + CONF.log_opt_values(LOG, logging.DEBUG) try: wsgi.serve_forever() diff --git a/ironic/cmd/conductor.py b/ironic/cmd/conductor.py index 5b38e57717..2867d3e2c3 100644 --- a/ironic/cmd/conductor.py +++ b/ironic/cmd/conductor.py @@ -26,7 +26,6 @@ from oslo_config import cfg from oslo_log import log from oslo_service import service -from ironic.common.i18n import _LI from ironic.common import service as ironic_service CONF = cfg.CONF @@ -41,8 +40,8 @@ def main(): 'ConductorManager') LOG = log.getLogger(__name__) - LOG.info(_LI("Configuration:")) - CONF.log_opt_values(LOG, logging.INFO) + LOG.debug("Configuration:") + CONF.log_opt_values(LOG, logging.DEBUG) launcher = service.launch(CONF, mgr) launcher.wait()