From 5260078db5744e48d2b282346e81d4ccdbe6bbae Mon Sep 17 00:00:00 2001 From: Filip Leonarski Date: Thu, 30 Oct 2025 16:56:40 +0100 Subject: [PATCH] COMMON: updated logger_config.py to include a test and to log to connect the correct logger --- common/src/aaredaqlib/logger_config.py | 25 ++++++++++++++++++++++--- 1 file changed, 22 insertions(+), 3 deletions(-) diff --git a/common/src/aaredaqlib/logger_config.py b/common/src/aaredaqlib/logger_config.py index 7744eb48..353dcc0a 100644 --- a/common/src/aaredaqlib/logger_config.py +++ b/common/src/aaredaqlib/logger_config.py @@ -20,7 +20,7 @@ class QtLogHandler(logging.Handler): except Exception: self.handleError(record) -def setup_logger(name="aaredaq", base_dir: str | None = '/tmp/mxlogs'): +def setup_logger(name="aareDAQ", base_dir: str | None = '/tmp/mxlogs'): # switch to production mode using: $ APP_ENV=prod python main.py env = os.getenv("APP_ENV", "dev") # default: dev config_file = ( @@ -34,12 +34,12 @@ def setup_logger(name="aaredaq", base_dir: str | None = '/tmp/mxlogs'): # Ensure directories for file handlers exist handlers = config.get("handlers", {}) + for h in handlers.values(): # RotatingFileHandler and FileHandler use "filename" filename = h.get("filename") if not filename: continue - if base_dir: abs_filename = os.path.join(base_dir, os.path.basename(filename)) else: @@ -55,7 +55,6 @@ def setup_logger(name="aaredaq", base_dir: str | None = '/tmp/mxlogs'): os.makedirs(log_dir, exist_ok=True) logging.config.dictConfig(config) - logging.getLogger("redis_lock").setLevel(logging.WARNING) logging.getLogger("urllib3").setLevel(logging.WARNING) logging.getLogger("matplotlib.font_manager").setLevel(logging.WARNING) @@ -80,5 +79,25 @@ def find_existing_formatter(default_fmt="%(asctime)s [%(levelname)s] %(name)s: % def attach_to_logger(logger_name: str = "", handler: logging.Handler = None): logging.getLogger(logger_name).addHandler(handler) +if __name__ == "__main__": + os.environ["APP_ENV"] = "dev" # or "prod" + #os.environ["LOG_DIR"] = "/tmp/mxlogs" # optional; matches your setup code + + log = setup_logger("aareDAQ", base_dir="/tmp/test_logs") # use the configured logger name + + log.debug("debug smoke test") + log.error("error smoke test") + log.info("info smoke test") + log.warning("warning smoke test") + log.critical("critical smoke test") + + for h in log.handlers: + if hasattr(h, "flush"): + h.flush() + + for h in log.handlers: + log.debug("debug test") + log.error("error test") + print(type(h), getattr(h, "baseFilename", None), h.level)