scrub process tags from debugger snapshots by vandonr · Pull Request #8406 · DataDog/dd-trace-dotnet

vandonr · 2026-04-02T12:49:07Z

Summary of changes

follow up on #8296
debugger itests didn't run in the PR
so we missed the fact that we need to apply the same scrubbing we did on other tests for those too

Reason for change

Implementation details

Test coverage

Other details

andrewlock

🤞

dd-trace-dotnet-ci-bot · 2026-04-02T13:35:41Z

Execution-Time Benchmarks Report ⏱️

Execution-time results for samples comparing This PR (8406) and master.

✅ No regressions detected - check the details below

Full Metrics Comparison

FakeDbCommand

Metric	Master (Mean ± 95% CI)	Current (Mean ± 95% CI)	Change	Status
.NET Framework 4.8 - Baseline
duration	72.27 ± (72.40 - 72.79) ms	75.31 ± (75.27 - 75.65) ms	+4.2%	✅⬆️
.NET Framework 4.8 - Bailout
duration	76.16 ± (76.05 - 76.40) ms	79.65 ± (79.45 - 79.83) ms	+4.6%	✅⬆️
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration	1095.57 ± (1093.81 - 1101.85) ms	1101.86 ± (1101.57 - 1107.69) ms	+0.6%	✅⬆️
.NET Core 3.1 - Baseline
process.internal_duration_ms	23.13 ± (23.07 - 23.19) ms	23.17 ± (23.12 - 23.23) ms	+0.2%	✅⬆️
process.time_to_main_ms	88.89 ± (88.69 - 89.09) ms	88.78 ± (88.58 - 88.98) ms	-0.1%	✅
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	10.91 ± (10.91 - 10.92) MB	10.93 ± (10.92 - 10.93) MB	+0.1%	✅⬆️
runtime.dotnet.threads.count	12 ± (12 - 12)	12 ± (12 - 12)	+0.0%	✅
.NET Core 3.1 - Bailout
process.internal_duration_ms	23.14 ± (23.09 - 23.19) ms	23.06 ± (22.99 - 23.12) ms	-0.4%	✅
process.time_to_main_ms	90.36 ± (90.14 - 90.59) ms	90.21 ± (89.99 - 90.43) ms	-0.2%	✅
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	10.95 ± (10.95 - 10.96) MB	10.95 ± (10.94 - 10.95) MB	-0.1%	✅
runtime.dotnet.threads.count	13 ± (13 - 13)	13 ± (13 - 13)	+0.0%	✅
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms	235.16 ± (234.14 - 236.17) ms	234.64 ± (233.33 - 235.95) ms	-0.2%	✅
process.time_to_main_ms	544.41 ± (543.08 - 545.73) ms	541.08 ± (539.67 - 542.50) ms	-0.6%	✅
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	47.82 ± (47.79 - 47.84) MB	47.90 ± (47.86 - 47.93) MB	+0.2%	✅⬆️
runtime.dotnet.threads.count	28 ± (28 - 29)	28 ± (28 - 28)	-0.4%	✅
.NET 6 - Baseline
process.internal_duration_ms	22.05 ± (22.00 - 22.10) ms	21.91 ± (21.86 - 21.95) ms	-0.6%	✅
process.time_to_main_ms	77.56 ± (77.35 - 77.76) ms	77.49 ± (77.30 - 77.67) ms	-0.1%	✅
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	10.63 ± (10.63 - 10.63) MB	10.64 ± (10.64 - 10.64) MB	+0.1%	✅⬆️
runtime.dotnet.threads.count	10 ± (10 - 10)	10 ± (10 - 10)	+0.0%	✅
.NET 6 - Bailout
process.internal_duration_ms	21.90 ± (21.84 - 21.95) ms	21.85 ± (21.80 - 21.91) ms	-0.2%	✅
process.time_to_main_ms	78.38 ± (78.20 - 78.55) ms	78.25 ± (78.04 - 78.46) ms	-0.2%	✅
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	10.70 ± (10.69 - 10.71) MB	10.75 ± (10.74 - 10.75) MB	+0.5%	✅⬆️
runtime.dotnet.threads.count	11 ± (11 - 11)	11 ± (11 - 11)	+0.0%	✅
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms	385.09 ± (382.71 - 387.47) ms	386.88 ± (384.66 - 389.09) ms	+0.5%	✅⬆️
process.time_to_main_ms	540.26 ± (539.26 - 541.27) ms	538.20 ± (537.22 - 539.18) ms	-0.4%	✅
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	49.84 ± (49.81 - 49.87) MB	49.74 ± (49.71 - 49.77) MB	-0.2%	✅
runtime.dotnet.threads.count	28 ± (28 - 28)	28 ± (28 - 28)	-0.0%	✅
.NET 8 - Baseline
process.internal_duration_ms	20.03 ± (19.98 - 20.08) ms	20.02 ± (19.97 - 20.07) ms	-0.1%	✅
process.time_to_main_ms	75.92 ± (75.75 - 76.08) ms	76.12 ± (75.96 - 76.29) ms	+0.3%	✅⬆️
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	7.66 ± (7.66 - 7.66) MB	7.67 ± (7.67 - 7.68) MB	+0.2%	✅⬆️
runtime.dotnet.threads.count	10 ± (10 - 10)	10 ± (10 - 10)	+0.0%	✅
.NET 8 - Bailout
process.internal_duration_ms	19.96 ± (19.91 - 20.01) ms	20.12 ± (20.06 - 20.19) ms	+0.8%	✅⬆️
process.time_to_main_ms	76.96 ± (76.75 - 77.17) ms	77.29 ± (77.08 - 77.50) ms	+0.4%	✅⬆️
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	7.73 ± (7.72 - 7.74) MB	7.73 ± (7.73 - 7.74) MB	+0.0%	✅⬆️
runtime.dotnet.threads.count	11 ± (11 - 11)	11 ± (11 - 11)	+0.0%	✅
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms	304.59 ± (302.32 - 306.85) ms	308.34 ± (306.00 - 310.68) ms	+1.2%	✅⬆️
process.time_to_main_ms	499.98 ± (499.16 - 500.79) ms	501.76 ± (500.98 - 502.54) ms	+0.4%	✅⬆️
runtime.dotnet.exceptions.count	0 ± (0 - 0)	0 ± (0 - 0)	+0.0%	✅
runtime.dotnet.mem.committed	37.10 ± (37.07 - 37.13) MB	37.09 ± (37.05 - 37.12) MB	-0.0%	✅
runtime.dotnet.threads.count	27 ± (27 - 27)	27 ± (27 - 27)	-0.1%	✅

HttpMessageHandler

Metric	Master (Mean ± 95% CI)	Current (Mean ± 95% CI)	Change	Status
.NET Framework 4.8 - Baseline
duration	213.57 ± (213.09 - 214.03) ms	212.25 ± (212.05 - 213.18) ms	-0.6%	✅
.NET Framework 4.8 - Bailout
duration	217.56 ± (217.20 - 218.23) ms	217.07 ± (216.59 - 217.55) ms	-0.2%	✅
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration	1219.98 ± (1221.50 - 1229.71) ms	1235.90 ± (1232.73 - 1239.44) ms	+1.3%	✅⬆️
.NET Core 3.1 - Baseline
process.internal_duration_ms	203.57 ± (203.06 - 204.08) ms	209.03 ± (208.50 - 209.55) ms	+2.7%	✅⬆️
process.time_to_main_ms	88.68 ± (88.44 - 88.91) ms	90.79 ± (90.56 - 91.02) ms	+2.4%	✅⬆️
runtime.dotnet.exceptions.count	3 ± (3 - 3)	3 ± (3 - 3)	+0.0%	✅
runtime.dotnet.mem.committed	15.96 ± (15.94 - 15.97) MB	15.92 ± (15.90 - 15.93) MB	-0.2%	✅
runtime.dotnet.threads.count	20 ± (20 - 20)	20 ± (20 - 20)	-0.0%	✅
.NET Core 3.1 - Bailout
process.internal_duration_ms	203.96 ± (203.44 - 204.47) ms	208.88 ± (208.33 - 209.43) ms	+2.4%	✅⬆️
process.time_to_main_ms	89.58 ± (89.38 - 89.78) ms	92.42 ± (92.16 - 92.68) ms	+3.2%	✅⬆️
runtime.dotnet.exceptions.count	3 ± (3 - 3)	3 ± (3 - 3)	+0.0%	✅
runtime.dotnet.mem.committed	15.99 ± (15.96 - 16.01) MB	15.95 ± (15.93 - 15.97) MB	-0.2%	✅
runtime.dotnet.threads.count	21 ± (21 - 21)	21 ± (21 - 21)	+1.2%	✅⬆️
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms	414.12 ± (412.70 - 415.54) ms	422.52 ± (420.98 - 424.05) ms	+2.0%	✅⬆️
process.time_to_main_ms	544.31 ± (542.89 - 545.73) ms	556.11 ± (554.79 - 557.43) ms	+2.2%	✅⬆️
runtime.dotnet.exceptions.count	3 ± (3 - 3)	3 ± (3 - 3)	+0.0%	✅
runtime.dotnet.mem.committed	58.67 ± (58.63 - 58.72) MB	58.73 ± (58.69 - 58.77) MB	+0.1%	✅⬆️
runtime.dotnet.threads.count	30 ± (30 - 30)	30 ± (30 - 30)	+0.6%	✅⬆️
.NET 6 - Baseline
process.internal_duration_ms	207.51 ± (206.97 - 208.05) ms	214.41 ± (213.81 - 215.02) ms	+3.3%	✅⬆️
process.time_to_main_ms	76.54 ± (76.30 - 76.78) ms	79.83 ± (79.54 - 80.12) ms	+4.3%	✅⬆️
runtime.dotnet.exceptions.count	4 ± (4 - 4)	4 ± (4 - 4)	+0.0%	✅
runtime.dotnet.mem.committed	16.23 ± (16.21 - 16.26) MB	16.19 ± (16.17 - 16.22) MB	-0.2%	✅
runtime.dotnet.threads.count	19 ± (19 - 19)	20 ± (20 - 20)	+1.1%	✅⬆️
.NET 6 - Bailout
process.internal_duration_ms	205.34 ± (204.82 - 205.85) ms	215.27 ± (214.76 - 215.79) ms	+4.8%	✅⬆️
process.time_to_main_ms	77.17 ± (76.95 - 77.38) ms	81.47 ± (81.18 - 81.77) ms	+5.6%	✅⬆️
runtime.dotnet.exceptions.count	4 ± (4 - 4)	4 ± (4 - 4)	+0.0%	✅
runtime.dotnet.mem.committed	16.28 ± (16.26 - 16.31) MB	16.17 ± (16.15 - 16.19) MB	-0.7%	✅
runtime.dotnet.threads.count	20 ± (20 - 20)	21 ± (20 - 21)	+2.0%	✅⬆️
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms	599.33 ± (596.69 - 601.97) ms	602.84 ± (600.27 - 605.40) ms	+0.6%	✅⬆️
process.time_to_main_ms	548.76 ± (547.58 - 549.94) ms	557.66 ± (556.32 - 559.01) ms	+1.6%	✅⬆️
runtime.dotnet.exceptions.count	4 ± (4 - 4)	4 ± (4 - 4)	+0.0%	✅
runtime.dotnet.mem.committed	61.38 ± (61.28 - 61.47) MB	61.20 ± (61.10 - 61.29) MB	-0.3%	✅
runtime.dotnet.threads.count	31 ± (31 - 31)	31 ± (31 - 31)	+0.3%	✅⬆️
.NET 8 - Baseline
process.internal_duration_ms	209.35 ± (208.84 - 209.86) ms	212.27 ± (211.75 - 212.80) ms	+1.4%	✅⬆️
process.time_to_main_ms	77.68 ± (77.42 - 77.95) ms	78.66 ± (78.43 - 78.88) ms	+1.3%	✅⬆️
runtime.dotnet.exceptions.count	4 ± (4 - 4)	4 ± (4 - 4)	+0.0%	✅
runtime.dotnet.mem.committed	11.61 ± (11.60 - 11.63) MB	11.61 ± (11.58 - 11.63) MB	-0.1%	✅
runtime.dotnet.threads.count	19 ± (19 - 19)	19 ± (19 - 19)	+0.4%	✅⬆️
.NET 8 - Bailout
process.internal_duration_ms	207.94 ± (207.39 - 208.49) ms	211.27 ± (210.83 - 211.71) ms	+1.6%	✅⬆️
process.time_to_main_ms	78.42 ± (78.13 - 78.71) ms	79.73 ± (79.55 - 79.92) ms	+1.7%	✅⬆️
runtime.dotnet.exceptions.count	4 ± (4 - 4)	4 ± (4 - 4)	+0.0%	✅
runtime.dotnet.mem.committed	11.69 ± (11.67 - 11.71) MB	11.67 ± (11.64 - 11.69) MB	-0.2%	✅
runtime.dotnet.threads.count	20 ± (20 - 20)	20 ± (20 - 20)	+1.5%	✅⬆️
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms	536.46 ± (530.34 - 542.57) ms	568.18 ± (560.53 - 575.82) ms	+5.9%	✅⬆️
process.time_to_main_ms	508.23 ± (507.17 - 509.29) ms	515.28 ± (514.32 - 516.25) ms	+1.4%	✅⬆️
runtime.dotnet.exceptions.count	4 ± (4 - 4)	4 ± (4 - 4)	+0.0%	✅
runtime.dotnet.mem.committed	50.77 ± (50.69 - 50.85) MB	51.18 ± (51.08 - 51.28) MB	+0.8%	✅⬆️
runtime.dotnet.threads.count	30 ± (30 - 30)	30 ± (30 - 30)	+0.3%	✅⬆️

Comparison explanation

Execution-time benchmarks measure the whole time it takes to execute a program, and are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are highlighted in **red**. The following thresholds were used for comparing the execution times:

Welch test with statistical test for significance of 5%
Only results indicating a difference greater than 5% and 5 ms are considered.

Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard.

Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph).

Duration charts

FakeDbCommand (.NET Framework 4.8)

gantt
    title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (75ms)  : 73, 78
    master - mean (73ms)  : 70, 76

    section Bailout
    This PR (8406) - mean (80ms)  : 78, 82
    master - mean (76ms)  : 75, 78

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (1,105ms)  : 1060, 1149
    master - mean (1,098ms)  : 1039, 1157

FakeDbCommand (.NET Core 3.1)

gantt
    title Execution time (ms) FakeDbCommand (.NET Core 3.1)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (119ms)  : 115, 123
    master - mean (119ms)  : 116, 123

    section Bailout
    This PR (8406) - mean (120ms)  : 118, 123
    master - mean (121ms)  : 117, 125

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (816ms)  : 796, 835
    master - mean (818ms)  : 795, 841

FakeDbCommand (.NET 6)

gantt
    title Execution time (ms) FakeDbCommand (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (106ms)  : 102, 111
    master - mean (106ms)  : 103, 110

    section Bailout
    This PR (8406) - mean (107ms)  : 104, 109
    master - mean (107ms)  : 105, 109

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (955ms)  : 919, 990
    master - mean (956ms)  : 918, 993

FakeDbCommand (.NET 8)

gantt
    title Execution time (ms) FakeDbCommand (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (104ms)  : 101, 107
    master - mean (104ms)  : 101, 107

    section Bailout
    This PR (8406) - mean (106ms)  : 102, 109
    master - mean (105ms)  : 103, 107

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (840ms)  : 801, 879
    master - mean (836ms)  : 801, 870

HttpMessageHandler (.NET Framework 4.8)

gantt
    title Execution time (ms) HttpMessageHandler (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (213ms)  : 207, 219
    master - mean (214ms)  : 207, 220

    section Bailout
    This PR (8406) - mean (217ms)  : 212, 222
    master - mean (218ms)  : 212, 223

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (1,236ms)  : 1185, 1287
    master - mean (1,226ms)  : 1164, 1287

HttpMessageHandler (.NET Core 3.1)

gantt
    title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (310ms)  : 301, 318
    master - mean (302ms)  : 293, 311

    section Bailout
    This PR (8406) - mean (311ms)  : 304, 317
    master - mean (303ms)  : 297, 309

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (1,020ms)  : 992, 1047
    master - mean (999ms)  : 974, 1024

HttpMessageHandler (.NET 6)

gantt
    title Execution time (ms) HttpMessageHandler (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (303ms)  : 295, 312
    master - mean (293ms)  : 285, 301

    section Bailout
    This PR (8406) - mean (306ms)  : 297, 316
    master - mean (291ms)  : 284, 299

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (1,191ms)  : 1150, 1232
    master - mean (1,179ms)  : 1140, 1219

HttpMessageHandler (.NET 8)

gantt
    title Execution time (ms) HttpMessageHandler (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8406) - mean (301ms)  : 293, 309
    master - mean (298ms)  : 290, 306

    section Bailout
    This PR (8406) - mean (302ms)  : 296, 307
    master - mean (297ms)  : 291, 304

    section CallTarget+Inlining+NGEN
    This PR (8406) - mean (1,119ms)  : 1007, 1232
    master - mean (1,080ms)  : 994, 1166

scrub process tags from debugger snapshots

9773cda

vandonr requested a review from a team as a code owner April 2, 2026 12:49

github-actions bot added the area:tests unit tests, integration tests label Apr 2, 2026

andrewlock approved these changes Apr 2, 2026

View reviewed changes

bouwkast approved these changes Apr 2, 2026

View reviewed changes

dudikeleti approved these changes Apr 2, 2026

View reviewed changes

vandonr enabled auto-merge (squash) April 2, 2026 14:32

vandonr merged commit 94d461d into master Apr 2, 2026
142 of 143 checks passed

vandonr deleted the vandonr/process3 branch April 2, 2026 15:00

github-actions bot added this to the vNext-v3 milestone Apr 2, 2026

vandonr mentioned this pull request Apr 2, 2026

Revert "switch process tags on by default (#8296)" #8405

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

scrub process tags from debugger snapshots#8406

scrub process tags from debugger snapshots#8406
vandonr merged 1 commit intomasterfrom
vandonr/process3

vandonr commented Apr 2, 2026

Uh oh!

andrewlock left a comment

Uh oh!

dd-trace-dotnet-ci-bot bot commented Apr 2, 2026

FakeDbCommand

HttpMessageHandler

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

vandonr commented Apr 2, 2026

Summary of changes

Reason for change

Implementation details

Test coverage

Other details

Uh oh!

andrewlock left a comment

Choose a reason for hiding this comment

Uh oh!

dd-trace-dotnet-ci-bot bot commented Apr 2, 2026

Execution-Time Benchmarks Report ⏱️

FakeDbCommand

HttpMessageHandler

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants