Open vitkyrka opened 3 months ago
[Fast Unit Tests Report]
On pipeline 37953089 (CI Visibility). The following jobs did not run any unit tests:
If you modified Go files and expected unit tests to run in these jobs, please double check the job logs. If you think tests should have been executed reach out to #agent-devx-help
Run ID: 0dc6b4cf-902c-4fc0-a9c8-b4720d59cf5b Metrics dashboard Target profiles
Baseline: 1248ec4f0bece8d46a33f20ff1447c7de7be6706 Comparison: 9c64dc75bb465e8650903c63ea992fb913a37b94
Performance changes are noted in the perf column of each table:
Confidence level: 90.00% Effect size tolerance: |Δ mean %| ≥ 5.00%
There were no significant changes in experiment optimization goals at this confidence level and effect size tolerance.
What does this PR do?
Flush batched events from uprobe hook in HTTP/2 and Kafka, similar to the way it is already done for HTTP. This is to ensure that the events are flushed in a timely manner even if the packet receive tracepoint does not hit on the CPU which queues the events.
Postgres is not handled in this PR since adding the flush there caused the code size limits to be exceeded on 4.14.
Motivation
To address flakiness in Kafka tests which generate a lot of events: https://datadoghq.atlassian.net/browse/USMON-1025
Additional Notes
Load test was run with
kafka.tls
andgrpc.golang.tls
.Possible Drawbacks / Trade-offs
Describe how to test/QA your changes