Update .gitlab/one-pipeline.locked.yaml#6818
Conversation
andrewlock
left a comment
There was a problem hiding this comment.
How does this interact with this include? https://github.com/DataDog/dd-trace-dotnet/blob/master/.gitlab-ci.yml#L11
andrewlock
left a comment
There was a problem hiding this comment.
OK, that makes more sense, thanks! 😄
Datadog ReportBranch report: ✅ 0 Failed, 249443 Passed, 2385 Skipped, 21h 15m 22.16s Total Time |
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 (6818) - mean (70ms) : 65, 75
. : milestone, 70,
master - mean (70ms) : 67, 72
. : milestone, 70,
section CallTarget+Inlining+NGEN
This PR (6818) - mean (1,010ms) : 991, 1029
. : milestone, 1010,
master - mean (1,007ms) : 984, 1030
. : milestone, 1007,
gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6818) - mean (103ms) : 101, 106
. : milestone, 103,
master - mean (103ms) : 100, 105
. : milestone, 103,
section CallTarget+Inlining+NGEN
This PR (6818) - mean (691ms) : 671, 710
. : milestone, 691,
master - mean (686ms) : 664, 708
. : milestone, 686,
gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6818) - mean (90ms) : 88, 92
. : milestone, 90,
master - mean (90ms) : 87, 92
. : milestone, 90,
section CallTarget+Inlining+NGEN
This PR (6818) - mean (649ms) : 635, 664
. : milestone, 649,
master - mean (644ms) : 630, 659
. : milestone, 644,
gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6818) - mean (190ms) : 187, 193
. : milestone, 190,
master - mean (190ms) : 187, 193
. : milestone, 190,
section CallTarget+Inlining+NGEN
This PR (6818) - mean (1,110ms) : 1083, 1136
. : milestone, 1110,
master - mean (1,108ms) : 1071, 1144
. : milestone, 1108,
gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6818) - mean (268ms) : 266, 271
. : milestone, 268,
master - mean (269ms) : 265, 273
. : milestone, 269,
section CallTarget+Inlining+NGEN
This PR (6818) - mean (875ms) : 839, 912
. : milestone, 875,
master - mean (872ms) : 844, 899
. : milestone, 872,
gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6818) - mean (261ms) : 256, 267
. : milestone, 261,
master - mean (262ms) : 257, 266
. : milestone, 262,
section CallTarget+Inlining+NGEN
This PR (6818) - mean (862ms) : 818, 905
. : milestone, 862,
master - mean (856ms) : 826, 887
. : milestone, 856,
|
andrewlock
left a comment
There was a problem hiding this comment.
Based on this, it looks like this is using https://github.com/DataDog/libdatadog-build/commit/a168809f1e2919f11f318db3df83667230e4d70a as we require, thanks
| @@ -0,0 +1,2 @@ | |||
| include: | |||
| - remote: https://gitlab-templates.ddbuild.io/libdatadog/one-pipeline/ca/05e116339b9780a138a474d0348e97debfca97f27bbc4ca489cf4e4c90d9cc94/one-pipeline.yml | |||
There was a problem hiding this comment.
Where does 05e116339b9780a138a474d0348e97debfca97f27bbc4ca489cf4e4c90d9cc94 come from? 🤔 How do we know how and when to update this value?
There was a problem hiding this comment.
This is something I need to document. Its a sha of all the files that are exported as part of one-pipeline.
The tooling will automatically detect when this needs to be updated in your Repository (e.g. only when the files change).
Benchmarks Report for tracer 🐌Benchmarks for #6818 compared to master:
The following thresholds were used for comparing the benchmark speeds:
Allocation changes below 0.5% are ignored. Benchmark detailsBenchmarks.Trace.ActivityBenchmark - Same speed ✔️ Same allocations ✔️Raw results
Benchmarks.Trace.AgentWriterBenchmark - 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 - Faster 🎉 Same allocations ✔️
|
| Benchmark | base/diff | Base Median (ns) | Diff Median (ns) | Modality |
|---|---|---|---|---|
| Benchmarks.Trace.SpanBenchmark.StartFinishScope‑net6.0 | 1.137 | 541.43 | 476.32 |
Raw results
| Branch | Method | Toolchain | Mean | StdError | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|---|---|---|---|---|---|---|---|---|---|
| master | StartFinishSpan |
net6.0 | 407ns | 0.179ns | 0.695ns | 0.00816 | 0 | 0 | 576 B |
| master | StartFinishSpan |
netcoreapp3.1 | 607ns | 0.299ns | 1.12ns | 0.0079 | 0 | 0 | 576 B |
| master | StartFinishSpan |
net472 | 655ns | 0.599ns | 2.32ns | 0.0918 | 0 | 0 | 578 B |
| master | StartFinishScope |
net6.0 | 541ns | 0.133ns | 0.498ns | 0.00978 | 0 | 0 | 696 B |
| master | StartFinishScope |
netcoreapp3.1 | 688ns | 0.456ns | 1.77ns | 0.00928 | 0 | 0 | 696 B |
| master | StartFinishScope |
net472 | 878ns | 0.235ns | 0.909ns | 0.104 | 0 | 0 | 658 B |
| #6818 | StartFinishSpan |
net6.0 | 433ns | 0.124ns | 0.482ns | 0.00801 | 0 | 0 | 576 B |
| #6818 | StartFinishSpan |
netcoreapp3.1 | 638ns | 0.649ns | 2.51ns | 0.00773 | 0 | 0 | 576 B |
| #6818 | StartFinishSpan |
net472 | 665ns | 0.579ns | 2.24ns | 0.0916 | 0 | 0 | 578 B |
| #6818 | StartFinishScope |
net6.0 | 476ns | 0.11ns | 0.425ns | 0.00965 | 0 | 0 | 696 B |
| #6818 | StartFinishScope |
netcoreapp3.1 | 728ns | 0.244ns | 0.946ns | 0.00944 | 0 | 0 | 696 B |
| #6818 | StartFinishScope |
net472 | 887ns | 0.256ns | 0.991ns | 0.104 | 0 | 0 | 658 B |
Benchmarks.Trace.TraceAnnotationsBenchmark - Faster 🎉 Same allocations ✔️
Faster 🎉 in #6818
Benchmark
base/diff
Base Median (ns)
Diff Median (ns)
Modality
Benchmarks.Trace.TraceAnnotationsBenchmark.RunOnMethodBegin‑net6.0
1.199
803.66
670.28
| Benchmark | base/diff | Base Median (ns) | Diff Median (ns) | Modality |
|---|---|---|---|---|
| Benchmarks.Trace.TraceAnnotationsBenchmark.RunOnMethodBegin‑net6.0 | 1.199 | 803.66 | 670.28 |
Raw results
| Branch | Method | Toolchain | Mean | StdError | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|---|---|---|---|---|---|---|---|---|---|
| master | RunOnMethodBegin |
net6.0 | 803ns | 0.404ns | 1.56ns | 0.00965 | 0 | 0 | 696 B |
| master | RunOnMethodBegin |
netcoreapp3.1 | 917ns | 0.537ns | 2.01ns | 0.00964 | 0 | 0 | 696 B |
| master | RunOnMethodBegin |
net472 | 1.14μs | 0.361ns | 1.4ns | 0.105 | 0 | 0 | 658 B |
| #6818 | RunOnMethodBegin |
net6.0 | 670ns | 0.315ns | 1.22ns | 0.00974 | 0 | 0 | 696 B |
| #6818 | RunOnMethodBegin |
netcoreapp3.1 | 903ns | 0.345ns | 1.33ns | 0.00924 | 0 | 0 | 696 B |
| #6818 | RunOnMethodBegin |
net472 | 1.08μs | 0.345ns | 1.29ns | 0.105 | 0 | 0 | 658 B |
Automatic PR. Updates .gitlab/one-pipeline.locked.yaml
Automatic PR. Updates .gitlab/one-pipeline.locked.yaml
Automatic PR. Updates .gitlab/one-pipeline.locked.yaml