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

Add OTLP tracing #668

Open
trajano opened this issue Nov 2, 2023 · 1 comment
Open

Add OTLP tracing #668

trajano opened this issue Nov 2, 2023 · 1 comment

Comments

@trajano
Copy link

trajano commented Nov 2, 2023

Summary

Allow tracing through OTLP compatible endpoints like Jaeger or Zipkin V3.

Steps to reproduce the problem

When starting a job from Cronicle

  • if it is HTTP start a span and have it traced to the OTLP endpoint so we can trace from the source and have it tagged with http
  • if it is a bash script provide the trace ID in an environment variable so the script do what it needs to do and have it tagged with bash
  • if it is the dummy plugin start a span and have it traced to the OTLP endpoint and have it tagged with dummy

Your Setup

Docker Swarm

Operating system and version?

Node.js version?

Cronicle software version?

Are you using a multi-server setup, or just a single server?

Are you using the filesystem as back-end storage, or S3/Couchbase?

Can you reproduce the crash consistently?

Log Excerpts

@trajano
Copy link
Author

trajano commented Nov 13, 2023

I think this https://opentelemetry.io/docs/instrumentation/js/automatic/ specifies how to do the automatic one which would be a good start but likely won't solve the actual problem because the plugins won't be tracing unless there's an automatic instrumentation created for plugins but that can be a different issue.

If you use the automation this step may not be needed https://opentelemetry.io/docs/instrumentation/js/manual/#initialize-the-sdk but adding it in here in case you need the SDK initialized to acquire the Tracer which will then be used to send the necessary spans.

Also ensure that you set the span kind to client if it is an HTTP.

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

No branches or pull requests

1 participant