Merge pull request #100 from dbmi-pitt/Derek-Furst/debug-config

added a configuration change allowing debug_mode to set the logging l…

Author: yuanzhou

src/app.py

@@ -25,8 +25,6 @@
 # Log rotation is handled via logrotate on the host system with a configuration file
 # Do NOT handle log file and rotation via the Python logging to avoid issues with multi-worker processes
 
-logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s:%(lineno)d: %(message)s', level=logging.DEBUG,
-                    datefmt='%Y-%m-%d %H:%M:%S')
 
 class SearchAPI:
     def __init__(self, config, translator_module, blueprint=None, ubkg_instance=None, progress_interface=None):
@@ -64,7 +62,10 @@ def __init__(self, config, translator_module, blueprint=None, ubkg_instance=None
         # If a Flask Blueprint is passed in from a service using this class, load that Blueprint.
         if blueprint is not None:
             self.app.register_blueprint(blueprint)
-
+        if self.DEBUG_MODE:
+            logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s: %(message)s', level=logging.DEBUG, datefmt='%Y-%m-%d %H:%M:%S')
+        else:
+            logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s: %(message)s', level=logging.INFO, datefmt='%Y-%m-%d %H:%M:%S')
         @self.app.errorhandler(400)
         def __http_bad_request(e):
             return self.http_bad_request(e)

src/indexer.py

@@ -3,8 +3,6 @@
 
 from libs.es_writer import ESWriter
 
-logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s: %(message)s', level=logging.INFO,
-                    datefmt='%Y-%m-%d %H:%M:%S')
 logger = logging.getLogger(__name__)
 
 class Indexer:

src/libs/es_writer.py

@@ -8,7 +8,6 @@
 # All the API logging is forwarded to the uWSGI server and gets written into the log file `uwsgo-entity-api.log`
 # Log rotation is handled via logrotate on the host system with a configuration file
 # Do NOT handle log file and rotation via the Python logging to avoid issues with multi-worker processes
-logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s:%(lineno)d: %(message)s', level=logging.DEBUG, datefmt='%Y-%m-%d %H:%M:%S')
 logger = logging.getLogger(__name__)
 
 class ESWriter:

src/opensearch_helper_functions.py

@@ -8,8 +8,6 @@
 from urllib.parse import urlparse
 from hubmap_commons.S3_worker import S3Worker
 
-logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s:%(lineno)d: %(message)s', level=logging.DEBUG,
-                    datefmt='%Y-%m-%d %H:%M:%S')
 logger = logging.getLogger(__name__)
 
 app = Flask(__name__, instance_path=os.path.join(os.path.abspath(os.path.dirname(__file__)), 'instance'),

src/translator/tranlation_helper_functions.py

@@ -5,8 +5,6 @@
 import yaml
 from flask import jsonify
 
-logging.basicConfig(format='[%(asctime)s] %(levelname)s in %(module)s: %(message)s', level=logging.DEBUG,
-                    datefmt='%Y-%m-%d %H:%M:%S')
 logger = logging.getLogger(__name__)