From cae3564ca0aada56a97763c0b87bb5570f88a3ec Mon Sep 17 00:00:00 2001
From: Dmitry Tantsur <dtantsur@redhat.com>
Date: Wed, 6 Apr 2016 19:24:25 +0200
Subject: [PATCH] Reduce amount of unhelpful debug logging in the API service

* Ignore DEBUG logging from keystone{auth,middleware}
* Log eventlet.wsgi events with a proper (default) logger name,
  so that DEBUG messages are also ignored
* Enable INFO logging for eventlet.wsgi.server, so that we still see summary
  of every request

Change-Id: I954a2b37bcedefb2c2b653b8ba26a030ffd7c5b7
---
 ironic/common/service.py                                   | 7 ++++---
 ironic/tests/unit/common/test_service.py                   | 6 ++----
 .../notes/logging-keystoneauth-9db7e56c54c2473d.yaml       | 6 ++++++
 3 files changed, 12 insertions(+), 7 deletions(-)
 create mode 100644 releasenotes/notes/logging-keystoneauth-9db7e56c54c2473d.yaml

diff --git a/ironic/common/service.py b/ironic/common/service.py
index cfd8a04543a..42c87d3dbbc 100644
--- a/ironic/common/service.py
+++ b/ironic/common/service.py
@@ -129,13 +129,15 @@ def prepare_service(argv=[]):
                                          'sqlalchemy=WARNING',
                                          'keystoneclient=INFO',
                                          'stevedore=INFO',
-                                         'eventlet.wsgi.server=WARNING',
+                                         'eventlet.wsgi.server=INFO',
                                          'iso8601=WARNING',
                                          'paramiko=WARNING',
                                          'requests=WARNING',
                                          'neutronclient=WARNING',
                                          'glanceclient=WARNING',
                                          'urllib3.connectionpool=WARNING',
+                                         'keystonemiddleware.auth_token=INFO',
+                                         'keystoneauth.session=INFO',
                                          ])
     config.parse_args(argv)
     log.setup(CONF, 'ironic')
@@ -168,8 +170,7 @@ class WSGIService(service.ServiceBase):
         self.server = wsgi.Server(CONF, name, self.app,
                                   host=CONF.api.host_ip,
                                   port=CONF.api.port,
-                                  use_ssl=use_ssl,
-                                  logger_name=name)
+                                  use_ssl=use_ssl)
 
     def start(self):
         """Start serving this service using loaded configuration.
diff --git a/ironic/tests/unit/common/test_service.py b/ironic/tests/unit/common/test_service.py
index 3f2707a47d1..a188b095266 100644
--- a/ironic/tests/unit/common/test_service.py
+++ b/ironic/tests/unit/common/test_service.py
@@ -32,8 +32,7 @@ class TestWSGIService(base.TestCase):
                                             test_service.app,
                                             host='0.0.0.0',
                                             port=6385,
-                                            use_ssl=False,
-                                            logger_name=service_name)
+                                            use_ssl=False)
 
     @mock.patch.object(service.wsgi, 'Server')
     def test_workers_set_correct_setting(self, wsgi_server):
@@ -64,5 +63,4 @@ class TestWSGIService(base.TestCase):
                                             srv.app,
                                             host='0.0.0.0',
                                             port=6385,
-                                            use_ssl=True,
-                                            logger_name=service_name)
+                                            use_ssl=True)
diff --git a/releasenotes/notes/logging-keystoneauth-9db7e56c54c2473d.yaml b/releasenotes/notes/logging-keystoneauth-9db7e56c54c2473d.yaml
new file mode 100644
index 00000000000..8e8a464f64d
--- /dev/null
+++ b/releasenotes/notes/logging-keystoneauth-9db7e56c54c2473d.yaml
@@ -0,0 +1,6 @@
+---
+other:
+  - Do not show DEBUG logging from keystoneauth and keystonemiddleware
+    by default.
+  - Log eventlet.wsgi.server events with a proper logger name and ignore DEBUG
+    logging by default.