-
Notifications
You must be signed in to change notification settings - Fork 145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add INTEGRATIONS system tests to Dotnet Tracer CI #4345
Conversation
Datadog ReportBranch report: ❄️ New Flaky Tests (1)
|
Benchmarks Report 🐌Benchmarks for #4345 compared to master:
The following thresholds were used for comparing the benchmark speeds:
Allocation changes below 0.5% are ignored. Benchmark detailsBenchmarks.Trace.AgentWriterBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.AppSecBodyBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.AspNetCoreBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.DbCommandBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.ElasticsearchBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.GraphQLBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.HttpClientBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.ILoggerBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.Log4netBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.NLogBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.RedisBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.SerilogBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.SpanBenchmark - Slower
|
Benchmark | diff/base | Base Median (ns) | Diff Median (ns) | Modality |
---|---|---|---|---|
Benchmarks.Trace.SpanBenchmark.StartFinishScope‑net6.0 | 1.128 | 483.85 | 545.62 |
Benchmark | base/diff | Base Median (ns) | Diff Median (ns) | Modality |
---|---|---|---|---|
Benchmarks.Trace.SpanBenchmark.StartFinishSpan‑net6.0 | 1.247 | 473.13 | 379.31 |
Raw results
Branch | Method | Toolchain | Mean | StdError | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
---|---|---|---|---|---|---|---|---|---|
master | StartFinishSpan |
net6.0 | 473ns | 0.1ns | 0.389ns | 0.00745 | 0 | 0 | 536 B |
master | StartFinishSpan |
netcoreapp3.1 | 630ns | 0.336ns | 1.3ns | 0.00726 | 0 | 0 | 536 B |
master | StartFinishSpan |
net472 | 592ns | 0.0956ns | 0.345ns | 0.0852 | 0 | 0 | 538 B |
master | StartFinishScope |
net6.0 | 484ns | 0.158ns | 0.589ns | 0.00933 | 0 | 0 | 656 B |
master | StartFinishScope |
netcoreapp3.1 | 710ns | 0.219ns | 0.847ns | 0.00878 | 0 | 0 | 656 B |
master | StartFinishScope |
net472 | 833ns | 0.248ns | 0.961ns | 0.0981 | 0 | 0 | 618 B |
#4345 | StartFinishSpan |
net6.0 | 379ns | 0.126ns | 0.473ns | 0.00756 | 0 | 0 | 536 B |
#4345 | StartFinishSpan |
netcoreapp3.1 | 612ns | 0.221ns | 0.828ns | 0.00713 | 0 | 0 | 536 B |
#4345 | StartFinishSpan |
net472 | 584ns | 0.204ns | 0.762ns | 0.0853 | 0 | 0 | 538 B |
#4345 | StartFinishScope |
net6.0 | 546ns | 0.433ns | 1.68ns | 0.00924 | 0 | 0 | 656 B |
#4345 | StartFinishScope |
netcoreapp3.1 | 664ns | 1.72ns | 6.67ns | 0.00912 | 0 | 0 | 656 B |
#4345 | StartFinishScope |
net472 | 794ns | 0.26ns | 0.936ns | 0.0978 | 0 | 0 | 618 B |
Benchmarks.Trace.TraceAnnotationsBenchmark - Same speed ✔️ Same allocations ✔️
Raw results
Branch | Method | Toolchain | Mean | StdError | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
---|---|---|---|---|---|---|---|---|---|
master | RunOnMethodBegin |
net6.0 | 507ns | 0.172ns | 0.665ns | 0.00911 | 0 | 0 | 656 B |
master | RunOnMethodBegin |
netcoreapp3.1 | 820ns | 0.389ns | 1.45ns | 0.00872 | 0 | 0 | 656 B |
master | RunOnMethodBegin |
net472 | 959ns | 0.348ns | 1.35ns | 0.0979 | 0 | 0 | 618 B |
#4345 | RunOnMethodBegin |
net6.0 | 518ns | 0.189ns | 0.706ns | 0.00926 | 0 | 0 | 656 B |
#4345 | RunOnMethodBegin |
netcoreapp3.1 | 777ns | 0.286ns | 0.991ns | 0.00902 | 0 | 0 | 656 B |
#4345 | RunOnMethodBegin |
net472 | 971ns | 0.156ns | 0.605ns | 0.0981 | 0 | 0 | 618 B |
Execution-Time Benchmarks Report ⏱️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 (4345) - mean (2,997ms) : 2897, 3097
. : milestone, 2997,
master - mean (3,046ms) : 2920, 3172
. : milestone, 3046,
section CallTarget+Inlining+NGEN
This PR (4345) - mean (3,813ms) : 3730, 3897
. : milestone, 3813,
master - mean (3,833ms) : 3728, 3937
. : milestone, 3833,
gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (4345) - mean (3,129ms) : 3031, 3226
. : milestone, 3129,
master - mean (3,127ms) : 3010, 3244
. : milestone, 3127,
section CallTarget+Inlining+NGEN
This PR (4345) - mean (3,643ms) : 3574, 3712
. : milestone, 3643,
master - mean (3,632ms) : 3544, 3720
. : milestone, 3632,
gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (4345) - mean (3,101ms) : 3008, 3193
. : milestone, 3101,
master - mean (3,104ms) : 2964, 3245
. : milestone, 3104,
section CallTarget+Inlining+NGEN
This PR (4345) - mean (3,595ms) : 3545, 3646
. : milestone, 3595,
master - mean (3,603ms) : 3520, 3685
. : milestone, 3603,
gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (4345) - mean (191ms) : 187, 194
. : milestone, 191,
master - mean (191ms) : 186, 195
. : milestone, 191,
section CallTarget+Inlining+NGEN
This PR (4345) - mean (1,096ms) : 1051, 1141
. : milestone, 1096,
master - mean (1,085ms) : 1049, 1121
. : milestone, 1085,
gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (4345) - mean (371ms) : 366, 376
. : milestone, 371,
master - mean (370ms) : 366, 374
. : milestone, 370,
section CallTarget+Inlining+NGEN
This PR (4345) - mean (1,160ms) : 1137, 1184
. : milestone, 1160,
master - mean (1,163ms) : 1126, 1199
. : milestone, 1163,
gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (4345) - mean (358ms) : 354, 362
. : milestone, 358,
master - mean (359ms) : 354, 365
. : milestone, 359,
section CallTarget+Inlining+NGEN
This PR (4345) - mean (1,122ms) : 1085, 1159
. : milestone, 1122,
master - mean (1,119ms) : 1095, 1143
. : milestone, 1119,
|
Throughput/Crank Report:zap: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 (4345) (11.202M) : 0, 11202277
master (11.090M) : 0, 11090030
benchmarks/2.32.0 (11.375M) : 0, 11374525
benchmarks/2.9.0 (10.962M) : 0, 10962162
section Automatic
This PR (4345) (7.745M) : 0, 7744850
master (7.769M) : 0, 7769343
benchmarks/2.32.0 (8.070M) : 0, 8070318
benchmarks/2.9.0 (7.961M) : 0, 7961307
section Trace stats
master (7.770M) : 0, 7769944
benchmarks/2.32.0 (8.090M) : 0, 8090308
section Manual
This PR (4345) (9.855M) : 0, 9854634
master (10.029M) : 0, 10028784
benchmarks/2.32.0 (10.358M) : 0, 10357559
section Manual + Automatic
This PR (4345) (7.395M) : 0, 7395209
master (7.362M) : 0, 7362416
benchmarks/2.32.0 (7.698M) : 0, 7697947
section Version Conflict
master (6.692M) : 0, 6691750
benchmarks/2.32.0 (6.996M) : 0, 6995640
gantt
title Throughput Linux arm64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (4345) (9.486M) : 0, 9485775
master (9.720M) : 0, 9720071
benchmarks/2.32.0 (9.619M) : 0, 9619353
benchmarks/2.9.0 (9.727M) : 0, 9726575
section Automatic
This PR (4345) (6.694M) : 0, 6693895
master (6.740M) : 0, 6740171
benchmarks/2.32.0 (6.682M) : 0, 6681889
section Trace stats
master (6.648M) : 0, 6647613
benchmarks/2.32.0 (6.899M) : 0, 6899182
section Manual
This PR (4345) (8.671M) : 0, 8670912
master (8.566M) : 0, 8565697
benchmarks/2.32.0 (8.545M) : 0, 8545110
section Manual + Automatic
This PR (4345) (6.513M) : 0, 6513034
master (6.424M) : 0, 6424293
benchmarks/2.32.0 (6.707M) : 0, 6707147
section Version Conflict
master (5.900M) : 0, 5899730
benchmarks/2.32.0 (6.004M) : 0, 6004290
gantt
title Throughput Windows x64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (4345) (10.366M) : 0, 10365614
master (10.384M) : 0, 10384309
benchmarks/2.32.0 (10.272M) : 0, 10272136
benchmarks/2.9.0 (10.841M) : 0, 10841230
section Automatic
This PR (4345) (7.540M) : 0, 7540312
master (7.421M) : 0, 7421270
benchmarks/2.32.0 (7.559M) : 0, 7559361
benchmarks/2.9.0 (7.862M) : 0, 7862268
section Trace stats
master (7.549M) : 0, 7548894
benchmarks/2.32.0 (7.562M) : 0, 7562409
section Manual
This PR (4345) (9.580M) : 0, 9580157
master (9.511M) : 0, 9510968
benchmarks/2.32.0 (9.530M) : 0, 9530406
section Manual + Automatic
This PR (4345) (7.286M) : 0, 7286318
master (7.324M) : 0, 7324257
benchmarks/2.32.0 (7.279M) : 0, 7279188
section Version Conflict
master (6.695M) : 0, 6694521
benchmarks/2.32.0 (6.827M) : 0, 6827118
gantt
title Throughput Linux x64 (ASM) (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (4345) (7.611M) : 0, 7610930
master (7.539M) : 0, 7538814
benchmarks/2.32.0 (7.327M) : 0, 7327005
benchmarks/2.9.0 (7.934M) : 0, 7933677
section No attack
This PR (4345) (2.190M) : 0, 2189600
master (2.195M) : 0, 2195229
benchmarks/2.32.0 (2.076M) : 0, 2076414
benchmarks/2.9.0 (3.287M) : 0, 3286770
section Attack
This PR (4345) (1.857M) : 0, 1856981
master (1.882M) : 0, 1882207
benchmarks/2.32.0 (1.786M) : 0, 1786036
benchmarks/2.9.0 (2.600M) : 0, 2600407
section Blocking
This PR (4345) (3.691M) : 0, 3690536
master (3.814M) : 0, 3813723
benchmarks/2.32.0 (3.490M) : 0, 3490178
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice thank you.
Summary of changes
Add the INTEGRATIONS scenario from System Tests to the Dotnet Tracer's CI.
Reason for change
Enabling INTEGRATIONS scenario allow us to catch potential breakages to integrations early for future PRs.
Implementation details
Test coverage
Other details