Closed andrewlock closed 2 weeks ago
Branch report: andrew/profiling/fix-ssi-tests
Commit report: e45d4da
Test service: dd-trace-dotnet
:white_check_mark: 0 Failed, 363427 Passed, 2362 Skipped, 18h 19m 28.43s Total Time
Execution-time results for samples comparing the following branches/commits:
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 shown in red. The following thresholds were used for comparing the execution times:
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).
gantt
title Execution time (ms) FakeDbCommand (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6016) - mean (70ms) : 67, 73
. : milestone, 70,
master - mean (71ms) : 67, 75
. : milestone, 71,
section CallTarget+Inlining+NGEN
This PR (6016) - mean (1,083ms) : 1063, 1103
. : milestone, 1083,
master - mean (1,085ms) : 1060, 1110
. : milestone, 1085,
gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6016) - mean (109ms) : 106, 112
. : milestone, 109,
master - mean (110ms) : 106, 114
. : milestone, 110,
section CallTarget+Inlining+NGEN
This PR (6016) - mean (760ms) : 741, 779
. : milestone, 760,
master - mean (759ms) : 741, 778
. : milestone, 759,
gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6016) - mean (93ms) : 90, 96
. : milestone, 93,
master - mean (93ms) : 89, 97
. : milestone, 93,
section CallTarget+Inlining+NGEN
This PR (6016) - mean (708ms) : 686, 731
. : milestone, 708,
master - mean (712ms) : 692, 732
. : milestone, 712,
gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6016) - mean (190ms) : 187, 193
. : milestone, 190,
master - mean (190ms) : 186, 194
. : milestone, 190,
section CallTarget+Inlining+NGEN
This PR (6016) - mean (1,161ms) : 1136, 1185
. : milestone, 1161,
master - mean (1,165ms) : 1135, 1194
. : milestone, 1165,
gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6016) - mean (276ms) : 272, 281
. : milestone, 276,
master - mean (276ms) : 272, 280
. : milestone, 276,
section CallTarget+Inlining+NGEN
This PR (6016) - mean (924ms) : 906, 942
. : milestone, 924,
master - mean (927ms) : 906, 947
. : milestone, 927,
gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6016) - mean (264ms) : 260, 269
. : milestone, 264,
master - mean (265ms) : 261, 268
. : milestone, 265,
section CallTarget+Inlining+NGEN
This PR (6016) - mean (907ms) : 880, 934
. : milestone, 907,
master - mean (905ms) : 876, 934
. : milestone, 905,
Throughput results for AspNetCoreSimpleController comparing the following branches/commits:
Cases where throughput results for the PR are worse than latest master (5% drop or greater), results are shown in red.
Note that these results are based on a single point-in-time result for each branch. For full results, see one of the many, many dashboards!
gantt
title Throughput Linux x64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (6016) (11.099M) : 0, 11098553
master (11.202M) : 0, 11202155
benchmarks/2.9.0 (11.142M) : 0, 11142344
section Automatic
This PR (6016) (7.301M) : 0, 7301012
master (7.437M) : 0, 7437019
benchmarks/2.9.0 (7.893M) : 0, 7893301
section Trace stats
master (7.824M) : 0, 7824309
section Manual
master (11.306M) : 0, 11305639
section Manual + Automatic
This PR (6016) (6.814M) : 0, 6814443
master (6.918M) : 0, 6917708
section DD_TRACE_ENABLED=0
master (10.325M) : 0, 10325290
gantt
title Throughput Linux arm64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (6016) (9.445M) : 0, 9445306
master (9.731M) : 0, 9730961
benchmarks/2.9.0 (9.585M) : 0, 9584514
section Automatic
This PR (6016) (6.656M) : 0, 6655520
master (6.575M) : 0, 6574988
section Trace stats
master (7.001M) : 0, 7000523
section Manual
master (9.504M) : 0, 9504255
section Manual + Automatic
This PR (6016) (6.214M) : 0, 6213539
master (6.217M) : 0, 6216600
section DD_TRACE_ENABLED=0
master (9.049M) : 0, 9048961
gantt
title Throughput Windows x64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (6016) (10.173M) : 0, 10173472
master (10.040M) : 0, 10040090
benchmarks/2.9.0 (10.094M) : 0, 10093850
section Automatic
This PR (6016) (6.751M) : 0, 6751294
master (6.783M) : 0, 6782944
benchmarks/2.9.0 (7.257M) : 0, 7257483
section Trace stats
master (7.323M) : 0, 7323194
section Manual
master (10.061M) : 0, 10060796
section Manual + Automatic
This PR (6016) (6.105M) : 0, 6104785
master (6.342M) : 0, 6342001
section DD_TRACE_ENABLED=0
master (9.357M) : 0, 9357055
Benchmarks for #6016 compared to master:
The following thresholds were used for comparing the benchmark speeds:
Allocation changes below 0.5% are ignored.
Summary of changes
Fix profiler integration tests in the SSI run
Reason for change
5240 changed the behaviour of the profiler in the presence of SSI variables. We have a scheduled run on master that specifically sets the variables. This breaks the profiler integration tests that depend on those variables.
Implementation details
Add an "environment restorer" attribute, which mirrors the one we use in the tracing integration tests. Ensures that the SSI tests are working in their expected environment
Test coverage
This is the test, but did a dedicated run too, and it works
Other details
Related to #6014