-
-
Notifications
You must be signed in to change notification settings - Fork 311
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
[bug] OpenTelemetry Trace hierarchy broken #3117
Comments
There's a |
Checked my traces and it seems that all the SQL query spans have a |
I tested with the testrig |
I tried running on my "production" setup with the env vars:
to see if the problem was caused by the queue filling up but spans were still missing like before so I guess it's something else. I guess the testrig and a production server have some sort of difference in the way they function or are set up? |
I just run it with (with an plain tempo): GTS_TRACING_ENABLED: "true"
GTS_TRACING_ENDPOINT: tempo.monitoring.svc:4317
GTS_TRACING_INSECURE_TRANSPORT: "true"
GTS_TRACING_TRANSPORT: grpc do you run the collector: |
Gotosocial uses opentelemetry-go for traces which has some settings that can be set through I also use Tempo with the following configuration: GTS_TRACING_ENABLED: "true"
GTS_TRACING_TRANSPORT: "http"
GTS_TRACING_ENDPOINT: "<redacted>:4318"
GTS_TRACING_INSECURE_TRANSPORT: "true" |
Describe the bug with a clear and concise description of what the bug is.
For Trace-ID is "X-Request-Id" used.
But OpenTelemetry and W3C use the "Traceparent"-Header for documentation see here:
the content is:
<version>-<trace-id>-<parent-span-id>-<trace-flag>
where the id has to be strapped.
What's your GoToSocial Version?
0.16.0
GoToSocial Arch
amd64 container
What happened?
Tracing not mapped with loadbalancer
What you expected to happen?
Tracing not mapped with loadbalancer
How to reproduce it?
No response
Anything else we need to know?
No response
The text was updated successfully, but these errors were encountered: