Skip to content

Conversation

@frankh
Copy link
Contributor

@frankh frankh commented Oct 19, 2025

Problem

we can't develop locally, Ben is very sad about this

Changes

add log-capture to the docker-compose file (can't be in the bin/start script as it needs to be in the docker-compose network with the otel collector)

update otel collector to collect docker logs and ship them to log-capture

for now bin/clickhouse-logs-init needs to be run manually after bin/start to create the clickhouse tables

Note that it captures to team 1

How did you test this code?

ran locally

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Changelog: (features only) Is this feature complete?

@frankh frankh requested a review from benjackwhite October 19, 2025 19:51
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.

5 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

@github-actions
Copy link
Contributor

github-actions bot commented Oct 19, 2025

Size Change: 0 B

Total Size: 3.3 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 3.3 MB

compressed-size-action

@frankh frankh force-pushed the frank/logs-local-dev branch 4 times, most recently from 4351fa5 to 5721834 Compare October 21, 2025 07:45
@frankh frankh requested a review from benjackwhite October 21, 2025 08:26
@frankh frankh force-pushed the frank/logs-local-dev branch from 40aa6dd to c326090 Compare October 21, 2025 19:50
@frankh frankh merged commit 51eccad into master Oct 22, 2025
169 of 171 checks passed
@frankh frankh deleted the frank/logs-local-dev branch October 22, 2025 08:14
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.

3 participants