electric-sql / electric

Sync little subsets of your Postgres data into local apps and services.
https://electric-sql.com
Apache License 2.0
6.2k stars 146 forks source link

feat(satellite,electric): Implement distributed tracing using OpenTelemetry in the client #1371

Closed alco closed 1 month ago

alco commented 3 months ago

This is a follow-up to https://github.com/electric-sql/electric/pull/1370.

Besides adding OT to the client, here we also implement the passing of client's tracing context to the server in RPC calls across the network boundary. The GH Actions workflows that run E2E tests are updated to enable the export of client traces to Honeycomb.io where each test gets its own ci.test_file attribute and a bunch of other CI-related attributes.

alco commented 3 months ago

@msfstef Can you help me out here with addressing Kevin's feedback?

I see there are merge conflicts but haven't rebased so as not to mess up code comments from Kevin.

KyleAMathews commented 1 month ago

👋 we've been working the last month on a rebuild of the Electric server over at a temporary repo https://github.com/electric-sql/electric-next/

You can read more about why we made the decision at https://next.electric-sql.com/about

We're really excited about all the new possibilities the new server brings and we hope you'll check it out soon and give us your feedback.

We're now moving the temporary repo back here. As part of that migration we're closing all the old issues and PRs. We really appreciate you taking the time to investigate and create this improvement!