Skip to content

fix: require logspark[json] to pull python-json-logger#148

Merged
Kydoimos97 merged 1 commit into
releasefrom
fix/logspark-json-extra
Jun 11, 2026
Merged

fix: require logspark[json] to pull python-json-logger#148
Kydoimos97 merged 1 commit into
releasefrom
fix/logspark-json-extra

Conversation

@Kydoimos97

Copy link
Copy Markdown
Contributor

Problem

LogSpark 0.10.2 (pinned in 6.0.5) requires python-json-logger for SparkJsonHandler. WrenchCL was pinning logspark>=0.10.2 without the [json] extra, so python-json-logger was not installed. Any ECS service running in deployment mode (which uses SparkJsonHandler for structured JSON logs) crashes on startup with:

ModuleNotFoundError: No module named 'pythonjsonlogger'
logspark.Types.Exceptions.MissingDependencyException: Missing required dependencies: python-json-logger

This caused model-input-builder-prep to exit code 1 inside the ai-pipeline Step Function, making every scheduled pipeline run fail. All other ECS services rebuilt at the same time will hit the same crash on their next invocation.

Fix

Changed logspark>=0.10.2 to logspark[json]>=0.10.2 so the JSON extra (and its python-json-logger transitive dependency) is always installed with WrenchCL.

End-user impact

Restores pipeline execution for all ECS services. Without this fix, every service rebuilt against WrenchCL 6.0.5 crashes at startup.

Copilot AI review requested due to automatic review settings June 11, 2026 00:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Kydoimos97 Kydoimos97 merged commit d83b597 into release Jun 11, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants