-
Notifications
You must be signed in to change notification settings - Fork 44
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
Export OpenTelemetry traces during deploy #584
base: master
Are you sure you want to change the base?
Conversation
@zowoq could you give this a try? |
Happy to test this but I'm not familiar with opentelemetry so I'll need to figure out how to wire it up to telegraf, seems to require more than just enabling the telegraf opentelemetry plugin. |
Does it require any specific config? As I said, just enabling the plugin doesn't seem to work? |
The telegraf plugin only supports gRPC and our Haskell library only exports binary payloads via HTTP. 🤦 One way around this would be to run an instance of the OpenTelemetry Collector. It speaks both protocols. |
You can use environment variables to change where and how the cachix client exports traces: https://opentelemetry.io/docs/concepts/sdk-configuration/otlp-exporter-configuration/. The default HTTP endpoint is AFAIK, it's recommended to run the collector locally on each machine and configure it to export to your remote datastore. |
My interest in this was that it would integrate into our existing monitoring, I am not going to run another service (either a central instance or locally on each machine) just to pull metrics from a single service. Thank you anyway, I appreciate the effort that went into implementing this PR. |
@zowoq, yeah, that makes sense. I wonder how much work it would be to get our end speak gRPC 🤔 |
Note that you would only need to run telemetry-collector daemon on the cachix deploy nodes that would then push to your telegraf instance. |
Telegraf is a collector that we run on each machine that is scraped by our prometheus instance. |
Resolves #562.
Haven't tested this yet.