Skip to content

Conversation

rossgray
Copy link
Contributor

@rossgray rossgray commented Oct 1, 2025

Problem

Currently, our external logs go to the UI, however they don't get written to stdout, which makes debugging more difficult.

Changes

Ensure external logs get produced to Kafka and to stdout.

How did you test this code?

Relying on automated tests.

@rossgray rossgray requested a review from a team October 1, 2025 14:38
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

9 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Contributor

@tomasfarias tomasfarias left a comment

Choose a reason for hiding this comment

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

This works 👍 We'll work on a single logger setup to clean this up in a later PR.

@rossgray rossgray merged commit 6187bc8 into master Oct 2, 2025
249 of 251 checks passed
@rossgray rossgray deleted the batch-exports-update-external-logging branch October 2, 2025 14:10
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