[Profiler] Fix flakiness due to Libraries Info Cache - #8997
Conversation
|
Bits Code status: ✅ Done Comment @DataDog to request changes |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 56202663da
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
BenchmarksBenchmark execution time: 2026-08-05 10:43:31 Comparing candidate commit 0b22fab in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 72 metrics, 0 unstable metrics, 64 known flaky benchmarks, 62 flaky benchmarks without significant changes.
|
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8997) and master. ✅ No regressions detected |
Summary of changes
Add timeout knob for
LibrariesInfoCacheReason for change
On
arm64arch, tests may randomly fail with this error message:The behavior is sound for a customer/production environment, but in CI, where we have less core and processing may take time. We should account for and make the
time to waitlonger.Implementation details
DD_INTERNAL_PROFILING_LIBRARIES_CACHE_START_TIMEOUTConfiguration.cppto read and populate its valueLibrariesInfoCacheTest coverage
Other details