Skip to content
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

Tracing and host metrics with Opentelemetry #151

Closed
wants to merge 16 commits into from

Conversation

balegas
Copy link
Contributor

@balegas balegas commented Jul 20, 2024

  • use otel collector to collect and export metric
  • add new docker-compose for telemetry services
  • Do something with statsd metrics collect and export metrics with openTelemetry collector
  • Add other exporters for local use
  • Learn what was done in our prev observability PR, which seems a lot more comprehensive
  • Need to document how to setup
  • Need to replace host metric collector with prometheus-based one
  • Discuss with team issue with opentelemetry_bandit

Closes electric-sql/electric#1472.

Copy link

netlify bot commented Jul 20, 2024

Deploy Preview for electric-next ready!

Name Link
🔨 Latest commit 87a9f6e
🔍 Latest deploy log https://app.netlify.com/sites/electric-next/deploys/66b229a4e7665000080819da
😎 Deploy Preview https://deploy-preview-151--electric-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@balegas balegas changed the title Added tracing with Opentelemetry Tracing and host metrics with Opentelemetry Jul 21, 2024
@balegas balegas force-pushed the balegas/observability branch 3 times, most recently from 1b1fefb to d1daaad Compare July 25, 2024 12:22
@alco alco force-pushed the balegas/observability branch 2 times, most recently from 83007ab to bd025fe Compare July 31, 2024 08:16
Copy link
Member

@alco alco left a comment

Choose a reason for hiding this comment

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

Looks great! Looking forward to a setup guide.


| VARIABLE| Description |
|---------------|------------|
| STATSD_HOST | The address of the StatsD server |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do we still need StatsD?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I remember: yes. Ilia's Benchmark stuff works with that

@balegas balegas marked this pull request as ready for review August 1, 2024 11:30
@alco alco force-pushed the balegas/observability branch from 05f0c39 to 87a9f6e Compare August 6, 2024 13:48
@alco
Copy link
Member

alco commented Aug 7, 2024

Moved to electric-sql/electric#1487.

@alco alco closed this Aug 7, 2024
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.

Add initial observability
4 participants