refactor(logging): use centralized logging helpers in consumers

Replace inline log-level resolution and handler creation in
lightspeed_stack.py and runners/uvicorn.py with calls to the shared
resolve_log_level() and create_log_handler() from log.py.

Remove the private _resolve_log_level() from uvicorn.py and the direct
RichHandler import from lightspeed_stack.py.

Signed-off-by: Major Hayden <major@redhat.com>

Author: major

src/lightspeed_stack.py

@@ -9,41 +9,31 @@
 import sys
 from argparse import ArgumentParser
 
-
-from rich.logging import RichHandler
-
-from log import get_logger
+from log import get_logger, resolve_log_level, create_log_handler
 from configuration import configuration
 from runners.uvicorn import start_uvicorn
 from runners.quota_scheduler import start_quota_scheduler
 from utils import schema_dumper
-from constants import LIGHTSPEED_STACK_LOG_LEVEL_ENV_VAR, DEFAULT_LOG_LEVEL
-
-# Read log level from environment variable with validation
-log_level_str = os.environ.get(LIGHTSPEED_STACK_LOG_LEVEL_ENV_VAR, DEFAULT_LOG_LEVEL)
-log_level = getattr(logging, log_level_str.upper(), None)
-if not isinstance(log_level, int):
-    print(
-        f"WARNING: Invalid log level '{log_level_str}', falling back to {DEFAULT_LOG_LEVEL}",
-        file=sys.stderr,
-    )
-    log_level = getattr(logging, DEFAULT_LOG_LEVEL)
+from constants import LIGHTSPEED_STACK_LOG_LEVEL_ENV_VAR
+
+# Resolve log level and handler from centralized logging utilities
+log_level = resolve_log_level()
 
-# RichHandler's columnar layout produces very narrow log output in containers
-# without a TTY (Rich falls back to 80 columns, columns consume ~40 of those).
-# Use a plain format when there's no terminal attached.
+# Configure root logger. basicConfig(force=True) is intentionally root-logger-specific.
+# RichHandler needs format="%(message)s" to prevent double-formatting by the root Formatter.
+handler = create_log_handler()
 if sys.stderr.isatty():
     logging.basicConfig(
         level=log_level,
         format="%(message)s",
         datefmt="[%X]",
-        handlers=[RichHandler()],
+        handlers=[handler],
         force=True,
     )
 else:
     logging.basicConfig(
         level=log_level,
-        format="%(asctime)s %(levelname)-8s %(name)s:%(lineno)d %(message)s",
+        handlers=[handler],
         force=True,
     )
 

src/runners/uvicorn.py

@@ -1,39 +1,15 @@
 """Uvicorn runner."""
 
 import logging
-import os
 
 import uvicorn
 
-from constants import DEFAULT_LOG_LEVEL, LIGHTSPEED_STACK_LOG_LEVEL_ENV_VAR
-from log import get_logger
+from log import get_logger, resolve_log_level
 from models.config import ServiceConfiguration
 
 logger = get_logger(__name__)
 
 
-def _resolve_log_level() -> int:
-    """Resolve the uvicorn log level from the environment.
-
-    Reads the LIGHTSPEED_STACK_LOG_LEVEL environment variable and converts it
-    to a Python logging level constant. Falls back to the default log level
-    when the variable is unset or contains an invalid value.
-
-    Returns:
-        The resolved logging level as an integer constant.
-    """
-    level_str = os.environ.get(LIGHTSPEED_STACK_LOG_LEVEL_ENV_VAR, DEFAULT_LOG_LEVEL)
-    level = getattr(logging, level_str.upper(), None)
-    if not isinstance(level, int):
-        logger.warning(
-            "Invalid log level '%s', falling back to %s",
-            level_str,
-            DEFAULT_LOG_LEVEL,
-        )
-        level = getattr(logging, DEFAULT_LOG_LEVEL)
-    return level
-
-
 def start_uvicorn(configuration: ServiceConfiguration) -> None:
     """Start the Uvicorn server using the provided service configuration.
 
@@ -43,7 +19,7 @@ def start_uvicorn(configuration: ServiceConfiguration) -> None:
         `tls_certificate_path`, and `tls_key_password`). TLS fields may be None
         and will be forwarded to uvicorn.run as provided.
     """
-    log_level = _resolve_log_level()
+    log_level = resolve_log_level()
     logger.info("Starting Uvicorn with log level %s", logging.getLevelName(log_level))
 
     # please note: