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.

Sign up for GitHub

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

Jump to bottom

Kafka instrumentation creates span for no messages #3367

Closed
lachmatt opened this issue Apr 12, 2024 · 1 comment
Closed

Kafka instrumentation creates span for no messages #3367

lachmatt opened this issue Apr 12, 2024 · 1 comment
Assignees
@lachmatt
Labels
bug Something isn't working
Milestone
1.6.0

Comments

@lachmatt
Copy link
Contributor

lachmatt commented Apr 12, 2024 •
edited
Loading

Bug Report

Symptom

When Consume is called without timeout specified (e.g either with CancellationToken, or with no params), when there are no messages read from a topic, spans will be generated, by default, every 100ms (default value of CancellationDelayMaxMs from ConsumerConfig).
This results in great number of spans generated.

Affected versions of autoinstrumentation

v1.3.0, v1.4.0, v1.5.0

Workaround

Disable kafka client instrumentation e.g by setting OTEL_DOTNET_AUTO_TRACES_KAFKA_INSTRUMENTATION_ENABLED to false

Fix merged and targeted for next release v1.5.1.
@lachmatt lachmatt added this to the 1.5.1 milestone Apr 12, 2024
@lachmatt lachmatt added the bug Something isn't working label Apr 12, 2024
@pellared pellared changed the title Kafka instrumentation bug Kafka instrumentation creates span for no messages Apr 12, 2024
@pellared
Copy link
Member

Fixed by #3363

@lachmatt lachmatt mentioned this issue Apr 25, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lachmatt lachmatt

Labels
bug Something isn't working
Projects
OpenTelemetry .NET Automatic Instrumentation
Status: Done/Cut
Milestone
1.6.0
Development

No branches or pull requests
2 participants
@pellared @lachmatt