]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Capture warnings into logs
authorEric Harney <eharney@redhat.com>
Thu, 25 Feb 2016 17:13:11 +0000 (12:13 -0500)
committerEric Harney <eharney@redhat.com>
Mon, 29 Feb 2016 20:56:10 +0000 (20:56 +0000)
Modules that issue warnings via the python
warnings module result in messages printed
to stderr.

This change results in them being captured
into Cinder logs instead.  This means that
logs from production environments contain
all of the relevant info for debugging
issues.

Change-Id: I16a31e1ef0df824dd513f43fd22d631fa647e004

cinder/cmd/api.py
cinder/cmd/backup.py
cinder/cmd/manage.py
cinder/cmd/scheduler.py
cinder/cmd/volume.py

index 9057883c5662d2af98bfe76459622c1bdf6c3305..7fe1cb56d5a5d1d8412b6f143a040b45af4cc05b 100644 (file)
@@ -20,6 +20,7 @@
 import eventlet
 eventlet.monkey_patch()
 
+import logging as python_logging
 import sys
 
 from cinder import objects
@@ -47,6 +48,7 @@ def main():
     CONF(sys.argv[1:], project='cinder',
          version=version.version_string())
     logging.setup(CONF, "cinder")
+    python_logging.captureWarnings(True)
     utils.monkey_patch()
 
     gmr.TextGuruMeditation.setup_autorun(version)
index cc265e535faad2c1d86c3c3dee46a8a903a74bb5..1fb8c1512a0cccbd537345a15e820644b28c61d9 100644 (file)
@@ -17,6 +17,7 @@
 
 """Starter script for Cinder Volume Backup."""
 
+import logging as python_logging
 import sys
 
 import eventlet
@@ -45,6 +46,7 @@ def main():
     CONF(sys.argv[1:], project='cinder',
          version=version.version_string())
     logging.setup(CONF, "cinder")
+    python_logging.captureWarnings(True)
     utils.monkey_patch()
     gmr.TextGuruMeditation.setup_autorun(version)
     server = service.Service.create(binary='cinder-backup')
index 5c8ab2bcb8426bd3ba520193d85aa21dece34ce4..0dec106f39687ff9ea6811612cc8b6ffd58b7cb6 100644 (file)
@@ -55,6 +55,7 @@
 from __future__ import print_function
 
 
+import logging as python_logging
 import os
 import sys
 
@@ -554,6 +555,7 @@ def main():
         CONF(sys.argv[1:], project='cinder',
              version=version.version_string())
         logging.setup(CONF, "cinder")
+        python_logging.captureWarnings(True)
     except cfg.ConfigDirNotFoundError as details:
         print(_("Invalid directory: %s") % details)
         sys.exit(2)
index bea47e2b63e53c4da4cdea0c624905e4bf96fa3e..9692fafe9f5e7002b5f8c2d91c2b02861b829f1a 100644 (file)
@@ -20,6 +20,7 @@
 import eventlet
 eventlet.monkey_patch()
 
+import logging as python_logging
 import sys
 
 from oslo_config import cfg
@@ -45,6 +46,7 @@ def main():
     CONF(sys.argv[1:], project='cinder',
          version=version.version_string())
     logging.setup(CONF, "cinder")
+    python_logging.captureWarnings(True)
     utils.monkey_patch()
     gmr.TextGuruMeditation.setup_autorun(version)
     server = service.Service.create(binary='cinder-scheduler')
index c9877aefeddf2dd4deb73ea2741449d8130fe4f4..a20344c6de3e7a545bfd8be7899c519f8665440d 100644 (file)
@@ -17,6 +17,7 @@
 
 """Starter script for Cinder Volume."""
 
+import logging as python_logging
 import os
 
 import eventlet
@@ -60,6 +61,7 @@ def main():
     CONF(sys.argv[1:], project='cinder',
          version=version.version_string())
     logging.setup(CONF, "cinder")
+    python_logging.captureWarnings(True)
     utils.monkey_patch()
     gmr.TextGuruMeditation.setup_autorun(version)
     launcher = service.get_launcher()