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

[Bug]: recursive tracing in all-in-one #4657

Closed
yurishkuro opened this issue Aug 13, 2023 · 0 comments · Fixed by #4662
Closed

[Bug]: recursive tracing in all-in-one #4657

yurishkuro opened this issue Aug 13, 2023 · 0 comments · Fixed by #4662
Labels

Comments

@yurishkuro
Copy link
Member

What happened?

image

Steps to reproduce

  1. run all-in-one
  2. reload screen a few times
  3. observe above trace

Expected behavior

In the past only read queries to the query-service were logged. The span above is the write part, where Jaeger submits tracing spans back to itself. We never instrument the write path as that creates recursive trace collection. We need to exclude the write endpoints from being traced by OTEL instrumentation.

cc @afzal442

Relevant log output

No response

Screenshot

No response

Additional context

No response

Jaeger backend version

No response

SDK

No response

Pipeline

No response

Stogage backend

No response

Operating system

No response

Deployment model

No response

Deployment configs

No response

@yurishkuro yurishkuro added the bug label Aug 13, 2023
yurishkuro added a commit that referenced this issue Aug 14, 2023
## Which problem is this PR solving?
- Resolves #4657

## Description of the changes
- use NoOp tracer provider in the OTLP receiver configuration

## How was this change tested?
- ran the UI, reloaded page many times, this endpoint no longer shows up

Signed-off-by: Yuri Shkuro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant