feat: Enhanced logging configuration

.pre-commit-config.yaml

@@ -21,6 +21,7 @@ repos:
   - id: mypy
     additional_dependencies:
       - typed_ast
+      - types-PyYAML
 
 - repo: https://github.com/timothycrosley/isort
   rev: 5.12.0

Dockerfile

@@ -11,11 +11,13 @@ RUN apt-get -qq update \
         python3-setuptools \
         python3-twisted \
         python3-wheel \
+        python3-yaml \
         python-pil \
         python-pip \
         python-setuptools \
         python-twisted \
         python-wheel \
+        python-yaml \
     && pip install vncdotool==1.0.0 \
     && pip3 install vncdotool \
     && rm -rf \

README.md

@@ -34,6 +34,7 @@ python utils/installation_test/vnc-install-ucs.py --vnc isala:2 --language deu -
 
 ```
 mkdir dump/
+VNCAUTOMATE_DEBUG=logging.yaml \
 VNCAUTOMATE_TMP=1 \
 python3 -m vncautomate.cli \
 	--log debug \

logging.ini

@@ -0,0 +1,35 @@
+[loggers]
+keys=root,pil
+
+[logger_root]
+level=DEBUG
+handlers=console,file
+
+[logger_pil]
+level=ERROR
+propagate=0
+qualname=PIL
+
+[handlers]
+keys=console,file
+
+[handler_console]
+class=StreamHandler
+formatter=brief
+level=INFO
+args=(sys.stdout,)
+
+[handler_file]
+class=FileHandler
+formatter=precise
+level=DEBUG
+args=("vnc-automate2.log", "w")
+
+[formatters]
+keys=brief,precise
+
+[formatter_brief]
+format=%(asctime)s %(message)s
+
+[formatter_precise]
+format=%(asctime)s %(levelname)s [%(name)s:%(module)s:%(funcName)s]: %(message)s

logging.yaml

@@ -0,0 +1,28 @@
+version: 1
+formatters:
+  brief:
+    format: "%(asctime)s %(message)s"
+  precise:
+    format: "%(asctime)s %(levelname)s [%(name)s:%(module)s:%(funcName)s]: %(message)s"
+handlers:
+  console:
+    class: logging.StreamHandler
+    formatter: brief
+    level: INFO
+    stream: ext://sys.stdout
+  file:
+    class: logging.FileHandler
+    formatter: precise
+    level: DEBUG
+    filename: vnc-automate3.log
+loggers:
+  PIL:
+    level: ERROR
+    propagate: False
+root:
+  level: DEBUG
+  handlers:
+    - console
+    - file
+#incremental: True
+#disable_existing_loggers: False

mypy.ini

@@ -8,4 +8,8 @@ exclude = (?x)(
     |^bdist/
     |^build/
     |^debian/
+    |^venv/
     )
+
+[mypy-yaml.*]
+ignore_missing_imports = True

setup.cfg

@@ -49,3 +49,4 @@ install_requires =
 
 [options.extras_require]
 lxml = lxml>=3,<4
+yaml = PyYAML

src/vncautomate/__init__.py

@@ -32,6 +32,8 @@
 #
 
 import logging
+import logging.config
+import os
 import sys
 from types import TracebackType  # noqa: F401
 from typing import Optional, Type  # noqa: F401
@@ -47,14 +49,31 @@
 
 def init_logger(debug_level="info"):
     # type: (str) -> None
+    debug = os.getenv("VNCAUTOMATE_DEBUG", "")
+    if debug and os.path.exists(debug):
+        if debug.endswith((".ini", ".cfg")):
+            logging.config.fileConfig(debug)
+        elif debug.endswith((".yaml", ".yml", ".json")):
+            import yaml
+
+            with open(debug, "r") as fd:
+                conf = yaml.safe_load(fd)
+
+            logging.config.dictConfig(conf)
+        else:
+            sys.exit("Unknown log configuration %r" % (debug,))
+
+        return
+
     try:
-        logging.basicConfig(
-            format="%(asctime)s %(levelname)s [%(name)s:%(module)s:%(funcName)s]: %(message)s",
-            level=getattr(logging, debug_level.upper()),
-        )
+        level = getattr(logging, debug_level.upper())
     except AttributeError:
-        logging.error('Given log level "%s" is unknown', debug_level)
-        sys.exit(1)
+        sys.exit("Unknown log level %r" % (debug_level,))
+
+    logging.basicConfig(
+        format="%(asctime)s %(levelname)s [%(name)s:%(module)s:%(funcName)s]: %(message)s",
+        level=level,
+    )
 
 
 def connect_vnc(host):