Move logging from conf.py to setup(app); Default WARNING level

example/conf.py

@@ -15,10 +15,9 @@
 import sys
 
 import mlx.coverity
-from mlx.coverity import __version__, coverity_logging
+from mlx.coverity import __version__
 import mlx.traceability
 from decouple import config
-import logging
 
 pkg_version = __version__
 
@@ -320,11 +319,3 @@
 TRACEABILITY_ITEM_ID_REGEX = r"([A-Z_]+-[A-Z0-9_]+)"
 TRACEABILITY_ITEM_RELINK = {}
 
-log_level = os.environ.get('LOGLEVEL', None)
-if log_level:
-    try:
-        numeric_level = getattr(logging, log_level.upper(), None)
-        coverity_logging.LOGGER.setLevel(level=numeric_level)
-    except:
-        raise ValueError(f"Invalid log level: {log_level}")
-

mlx/coverity/coverity.py

@@ -8,12 +8,15 @@
 """
 
 from getpass import getpass
+import logging
+import os
+from sphinx.util.logging import getLogger
 from urllib.error import URLError, HTTPError
 
 from docutils import nodes
 
 from .__coverity_version__ import __version__
-from .coverity_logging import report_info, report_warning
+from .coverity_logging import report_info, report_warning, LOGGER
 from .coverity_services import CoverityDefectService
 from .coverity_directives.coverity_defect_list import (
     CoverityDefect,
@@ -159,6 +162,13 @@ def get_filtered_defects(self, node):
 # Extension setup
 def setup(app):
     """Extension setup"""
+    log_level = os.environ.get('LOGLEVEL', "WARNING")
+    try:
+        numeric_level = getattr(logging, log_level.upper())
+        LOGGER.setLevel(numeric_level)
+    except:
+        raise ValueError(f"Invalid log level: {log_level}")
+
     # Create default configuration. Can be customized in conf.py
     app.add_config_value(
         "coverity_credentials",

mlx/coverity/coverity_logging.py

@@ -1,10 +1,8 @@
 """Module to provide functions that accommodate logging."""
 
 from sphinx.util.logging import getLogger
-from logging import WARNING
 
 LOGGER = getLogger(__name__)
-LOGGER.setLevel(WARNING)
 
 
 def report_warning(msg, docname, lineno=None):