feat(ci): test binaries less heavy and less link time - #2383
feat(ci): test binaries less heavy and less link time#2383gh-worker-dd-mergequeue-cf854d[bot] merged 3 commits into
Conversation
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: 051e9c4 | Docs | View more details | Give us feedback! |
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
BenchmarksComparisonBenchmark execution time: 2026-08-20 15:36:26 Comparing candidate commit 051e9c4 in PR branch Found 7 performance improvements and 0 performance regressions! Performance is the same for 161 metrics, 10 unstable metrics.
|
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
What does this PR do?
Test binaries inherit [profile.dev] debug = 2 (full debug info) from root Cargo.toml.
Motivation
This dominated Windows link/PDB time and cached target size
Additional Notes
line-tables-onlykeeps file/line info for backtraces and llvm-cov.How to test the change?
On the first commit of this PR, it took 38min, which is pretty much in line with before the change, but cache needed rebuilding, I pushed an empty commit and it took less than 20 minutes, it might be a fluke, I'll do another empty commit, but it might be really big improvement for not much.
Apparently though, having a warm cache is pretty hard, and the 10gb repo limit is exceeded and cache evicted extremely quickly. Once caching is done by sccache, we would have the full upside of this, and it could have a real impact