8000 feat(capture): Set up capture and capture-relay for local dev by eli-r-ph · Pull Request #33819 · PostHog/posthog · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(capture): Set up capture and capture-relay for local dev #33819

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.

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

Merged
merged 2 commits into from
Jun 17, 2025

Conversation

eli-r-ph
Copy link
Contributor
@eli-r-ph eli-r-ph commented Jun 17, 2025

Problem

As part of migrating off capture.py we'll need to stand up the replacement Rust services in our local dev env.

Changes

Part 1 of a series:

  • Sets up capture and capture-relay to run as part of bin/start services
  • Swaps cymbal in bin/start to consume new exceptions topic, starts healthy now
  • Includes misc. related small cleanups and tweaks to the backing scripts

Did you write or update any docs for this change?

How did you test this code?

Local dev + CI

@eli-r-ph eli-r-ph requested a review from a team June 17, 2025 19:08
@eli-r-ph eli-r-ph self-assigned this Jun 17, 2025
@eli-r-ph eli-r-ph changed the title feat(capture): Add capture and capture-relay to bin/start feat(capture): Set up capture and capture-relay for local dev Jun 17, 2025
Copy link
Contributor
github-actions bot commented Jun 17, 2025

Size Change: 0 B

Total Size: 2.57 MB

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

compressed-size-action

@eli-r-ph eli-r-ph marked this pull request as ready for review June 17, 2025 21:00
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.

PR Summary

Added support for new Rust-based capture services in the local development environment, replacing the Python-based capture.py implementation.

  • Modified bin/start-rust-service to configure 'capture' and 'capture-relay' services with distinct ports and environment variables
  • Added rdkafka logging configuration to reduce noise in development logs
  • Updated bin/mprocs.yaml to include new services and streamlined database check preconditions
  • Added support for service name overrides in bin/start-rust-service to allow single binary to run in different modes
  • Updated 'cymbal' service to consume from new exceptions topic

2 files reviewed, no comments
Edit PR Review Bot Settings | Greptile

@eli-r-ph eli-r-ph merged commit ff8141a into master Jun 17, 2025
106 checks passed
@eli-r-ph eli-r-ph deleted the eli.r/add-capture-svcs-to-bin-start branch June 17, 2025 21:22
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
0