From 46b9a88ba110e5489749327f6dd381a8af695c3e Mon Sep 17 00:00:00 2001
From: Mark McLoughlin <markmc@redhat.com>
Date: Mon, 27 May 2013 19:42:57 +0100
Subject: Don't use pecan to configure logging

Fixes bug #1184941

openstack.common.log gives a mechanism to configure logging and, by
default, we will set up a handler on root logger. Using pecan to also
configure a handler on the 'ironic' logger means every log message gets
logged to console twice.

It seems that the pecan logging configuration stuff is redundant with
our existing logging configuration infrastructure.

Change-Id: Iec0630b0e842e3b6a90021bf552d9b857c5a0552
---
 ironic/api/app.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'ironic/api/app.py')

diff --git a/ironic/api/app.py b/ironic/api/app.py
index 8d9f1df50..b3e623715 100644
--- a/ironic/api/app.py
+++ b/ironic/api/app.py
@@ -58,8 +58,7 @@ def setup_app(pecan_config=None, extra_hooks=None):
         pecan_config.app.root,
         static_root=pecan_config.app.static_root,
         template_path=pecan_config.app.template_path,
-        logging=getattr(pecan_config, 'logging', {}),
-        debug=getattr(pecan_config.app, 'debug', False),
+        debug=CONF.debug,
         force_canonical=getattr(pecan_config.app, 'force_canonical', True),
         hooks=app_hooks,
     )
@@ -73,7 +72,6 @@ def setup_app(pecan_config=None, extra_hooks=None):
 class VersionSelectorApplication(object):
     def __init__(self):
         pc = get_pecan_config()
-        pc.app.debug = CONF.debug
         pc.app.enable_acl = (CONF.auth_strategy == 'keystone')
         self.v1 = setup_app(pecan_config=pc)
 
-- 
cgit v1.2.1