test(crashtracking): add minimal LD preload test for crashtracker collector#1428
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1428 +/- ##
==========================================
+ Coverage 70.96% 71.03% +0.07%
==========================================
Files 421 422 +1
Lines 68438 68697 +259
==========================================
+ Hits 48567 48802 +235
- Misses 19871 19895 +24
🚀 New features to boost your workflow:
|
BenchmarksComparisonBenchmark execution time: 2026-01-29 16:41:12 Comparing candidate commit ce1d7db in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
Group 16
Group 17
Group 18
Group 19
BaselineOmitted due to size. |
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-apple-darwin
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-apple-darwin
x86_64-unknown-linux-gnu
|
c9a32e0 to
5d2087f
Compare
e2ed49c to
677bb4f
Compare
c8ebe5f to
93c78fb
Compare
6fed733 to
ce1d7db
Compare
📚 Documentation Check Results📦
|
🔒 Cargo Deny Results📦
|
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
…cripts (#1506) # What does this PR do? Attempt to fix a potential race condition with the [all-checks](https://github.com/DataDog/libdatadog/blob/b44bb77dc7e7dcfd8e47d9e8c2bbe1d3cfa894f6/.github/workflows/all-checks.yml) workflow and [report_gitlab_CI_status](https://github.com/DataDog/libdatadog/blob/b44bb77dc7e7dcfd8e47d9e8c2bbe1d3cfa894f6/.gitlab-ci.yml#L42) `report_gitlab_CI_status` is `post` stage and doesn't get created until after dependent jobs run. If all gitlab benchmark and github actions successfully complete, but `report_gitlab_CI_status` hasn't been created yet, then all-checks can pass because all-checks doesn't see anything pending. The `dd-gitlab/default-pipeline` check has been enabled for this repo by ci-infra. This should solve our problems and remove the need for custom scripts. It will pass when the Gitlab pipeline completes. This check will be added to the branch protection rule for main. all-checks will just be used for github steps. # Motivation all-checks passed on #1428 , despite tests failing on the alpine builds in gitlab. # Additional Notes Anything else we should know when reviewing? # How to test the change? Describe here in detail how the change can be validated. Co-authored-by: edmund.kump <edmund.kump@datadoghq.com>

What does this PR do?
Motivation
We want to validate all uses of non-signal-safe functions in the handler.
How to test the change?
You can run the
test_collector_no_allocations_stacktrace_modestest. You can also add allocations in the signal hander path and observe the test behavior