-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Datadog Connector Memory Issues #29755
Comments
…30085) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Adds feature gate `connector.datadogconnector.performance` that can be used optionally to reduce memory footprint of datadog connector. **Link to tracking Issue:** #29755 **Testing:** <Describe what testing was performed and which tests were added.> - Tested internally using client data. **Documentation:** <Describe the documentation added.>
…pen-telemetry#30085) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Adds feature gate `connector.datadogconnector.performance` that can be used optionally to reduce memory footprint of datadog connector. **Link to tracking Issue:** open-telemetry#29755 **Testing:** <Describe what testing was performed and which tests were added.> - Tested internally using client data. **Documentation:** <Describe the documentation added.>
This is resolved via feature gate. See datadog connector readme. |
Hi, Wer'e still seeing memory issues, even with the feature gate enabled. |
Hi @grzn, Can you please give me an example on how do you enable the feature gate? I was looking for an example on how to do that but I didn't find anything. |
Same for us. We've reported our issue directly to DataDog. |
It's a command-line parameter to the binary. We were in v0.7something and it was all good. Now we're trying 0.92 and it's leaking. Going to try 0.82 which is the last version before the processor refactor. |
@grzn We didn't have success with the deprecated processor because it does not support computing stats by peer service and span kind. Once we enable it, we lost the ability to see metrics for inferred services. |
I'll push this through our DataDog channels as well. |
Cross-referencing to #30828 |
Component(s)
connector/datadog
What happened?
Description
Customers using Datadog connector at scale have reported Collector memory issues. We are able to replicate the issue with the help of trace dump . The collector using Datadog connector increases memory and OOMs within a few minutes of starting.
Steps to Reproduce
Use the collector config and send the traces down the pipe.
Expected Result
Collector shouldn't OOM.
Actual Result
Collector memory and CPU spike and we are unable to use Datadog Connector at scale.
Collector version
v0.91.0
Environment information
Environment
Latest GKE cluster.
OpenTelemetry Collector configuration
Log output
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: