From 7fd9886edd25ca752408d2884d61892cf6f60981 Mon Sep 17 00:00:00 2001 From: Ariel Schulz Date: Tue, 14 Apr 2026 13:53:45 +0200 Subject: [PATCH 1/3] Adapt structlog configuration to be more human-friendly * Switch timestamp from ISO to Y-m-d HH:MM:SS * Get rid of padding after log level * Remove callsite information; not useful for user --- exasol/toolbox/__init__.py | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/exasol/toolbox/__init__.py b/exasol/toolbox/__init__.py index efbda7a1f..e5276f400 100644 --- a/exasol/toolbox/__init__.py +++ b/exasol/toolbox/__init__.py @@ -4,8 +4,6 @@ import structlog from structlog.dev import ConsoleRenderer from structlog.processors import ( - CallsiteParameter, - CallsiteParameterAdder, TimeStamper, add_log_level, format_exc_info, @@ -18,16 +16,10 @@ processors=[ # 1. Enrich the data first add_log_level, - TimeStamper(fmt="iso"), - CallsiteParameterAdder( - { - CallsiteParameter.MODULE, - CallsiteParameter.FUNC_NAME, - } - ), + TimeStamper(fmt="%Y-%m-%d %H:%M:%S"), # 2. Handle exceptions format_exc_info, - # 3. Rendering option - ConsoleRenderer(), + # 3. Render + ConsoleRenderer(pad_level=False), ], ) From bc3921a2c475f3b42df590d3c7b2832f64516ddf Mon Sep 17 00:00:00 2001 From: Ariel Schulz Date: Tue, 14 Apr 2026 14:12:34 +0200 Subject: [PATCH 2/3] Switch so subsystem is only on the debug level --- exasol/toolbox/util/workflows/__init__.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/exasol/toolbox/util/workflows/__init__.py b/exasol/toolbox/util/workflows/__init__.py index 6eb60c176..ab6a48248 100644 --- a/exasol/toolbox/util/workflows/__init__.py +++ b/exasol/toolbox/util/workflows/__init__.py @@ -1,3 +1,8 @@ import structlog -logger = structlog.get_logger(__name__).bind(subsystem="workflows") +from exasol.toolbox import log_level + +logger = structlog.get_logger(__name__) + +if log_level == "DEBUG": + logger = logger.bind(subsystem="workflows") From 7a5be3c5a74686d5d0494bedb29d95d097210cf9 Mon Sep 17 00:00:00 2001 From: Ariel Schulz Date: Tue, 14 Apr 2026 14:14:48 +0200 Subject: [PATCH 3/3] Switch 'Load workflow template' to debug --- doc/changes/unreleased.md | 4 ++++ exasol/toolbox/util/workflows/workflow.py | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/doc/changes/unreleased.md b/doc/changes/unreleased.md index a4e73a370..eb8a0b83e 100644 --- a/doc/changes/unreleased.md +++ b/doc/changes/unreleased.md @@ -19,3 +19,7 @@ as described in the [Pull Request description](https://exasol.github.io/python-t ## Bugfix * #766: Fixed `action/upload-pages-artifact` from v5 to v5.0.0 + +## Feature + +* #733: Adjusted structlog and log level for workflow generation diff --git a/exasol/toolbox/util/workflows/workflow.py b/exasol/toolbox/util/workflows/workflow.py index 69b9e9654..92dfdef06 100644 --- a/exasol/toolbox/util/workflows/workflow.py +++ b/exasol/toolbox/util/workflows/workflow.py @@ -47,7 +47,7 @@ def load_from_template( patch_yaml: WorkflowCommentedMap | None = None, ): with bound_contextvars(template_file_name=file_path.name): - logger.info("Load workflow template: %s", file_path.name) + logger.debug("Load workflow template: %s", file_path.name) if not file_path.exists(): raise FileNotFoundError(file_path)