Skip to content

Update migration log group to pipe out json #180

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
abachman-dsac merged 1 commit into from
Nov 6, 2025
Merged

Update migration log group to pipe out json #180

abachman-dsac merged 1 commit into from
Nov 6, 2025

Conversation

@wbprice
Copy link
Contributor

@wbprice wbprice commented Nov 3, 2025 •
edited
Loading

  • Update migration image logging output structured json logs
  • Update dagster log group names to use format specified by the production readiness checklist for splunk.

@wbprice wbprice marked this pull request as ready for review November 4, 2025 15:23
abachman-dsac
abachman-dsac approved these changes Nov 6, 2025
View reviewed changes
Copy link
Member

@abachman-dsac abachman-dsac left a comment

Choose a reason for hiding this comment

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

STRUCTURE ALL THE THINGS 🧹 ✋

@abachman-dsac abachman-dsac merged commit 02770eb into main Nov 6, 2025
9 checks passed
@abachman-dsac abachman-dsac deleted the wbprice/update-dagster-log-group-name branch November 6, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abachman-dsac abachman-dsac abachman-dsac approved these changes

@dturner5234 dturner5234 Awaiting requested review from dturner5234

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@wbprice @abachman-dsac @acme-cms-challenge