[CI] Add CultureInvariant to a test that was failing locally#8127
[CI] Add CultureInvariant to a test that was failing locally#8127pierotibou merged 1 commit intomasterfrom
Conversation
Signed-off-by: Pierre Bonet <pierre.bonet@datadoghq.com>
...adog.Trace.ClrProfiler.Managed.Tests/AutoInstrumentation/MongoDb/TruncatedTextWriterTests.cs
Show resolved
Hide resolved
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8127) and master. ✅ No regressions detected - check the details below Full Metrics ComparisonFakeDbCommand
HttpMessageHandler
Comparison explanationExecution-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:
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 chartsFakeDbCommand (.NET Framework 4.8)gantt
title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8127) - mean (73ms) : 70, 75
master - mean (73ms) : 71, 75
section Bailout
This PR (8127) - mean (77ms) : 75, 79
master - mean (78ms) : 76, 79
section CallTarget+Inlining+NGEN
This PR (8127) - mean (1,058ms) : 1012, 1104
master - mean (1,066ms) : 1013, 1119
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 (8127) - mean (123ms) : 120, 126
master - mean (125ms) : 120, 129
section Bailout
This PR (8127) - mean (124ms) : 121, 127
master - mean (126ms) : 123, 129
section CallTarget+Inlining+NGEN
This PR (8127) - mean (828ms) : 772, 885
master - mean (833ms) : 780, 886
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8127) - mean (109ms) : 104, 114
master - mean (102ms) : 99, 106
section Bailout
This PR (8127) - mean (109ms) : crit, 107, 112
master - mean (103ms) : 100, 106
section CallTarget+Inlining+NGEN
This PR (8127) - mean (798ms) : crit, 775, 821
master - mean (742ms) : 666, 817
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8127) - mean (107ms) : 105, 110
master - mean (108ms) : 105, 110
section Bailout
This PR (8127) - mean (108ms) : 106, 110
master - mean (108ms) : 106, 110
section CallTarget+Inlining+NGEN
This PR (8127) - mean (713ms) : 696, 729
master - mean (712ms) : 690, 735
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 (8127) - mean (199ms) : 192, 205
master - mean (198ms) : 192, 204
section Bailout
This PR (8127) - mean (203ms) : 195, 210
master - mean (201ms) : 198, 205
section CallTarget+Inlining+NGEN
This PR (8127) - mean (1,193ms) : 1034, 1352
master - mean (1,170ms) : 1104, 1235
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 (8127) - mean (297ms) : 287, 306
master - mean (285ms) : 279, 291
section Bailout
This PR (8127) - mean (297ms) : 292, 302
master - mean (285ms) : 278, 293
section CallTarget+Inlining+NGEN
This PR (8127) - mean (1,006ms) : 962, 1050
master - mean (960ms) : 887, 1033
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8127) - mean (289ms) : 279, 298
master - mean (289ms) : 280, 299
section Bailout
This PR (8127) - mean (287ms) : 283, 292
master - mean (289ms) : 285, 294
section CallTarget+Inlining+NGEN
This PR (8127) - mean (995ms) : 958, 1031
master - mean (998ms) : 959, 1038
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8127) - mean (290ms) : 279, 301
master - mean (288ms) : 281, 295
section Bailout
This PR (8127) - mean (291ms) : 284, 297
master - mean (289ms) : 282, 295
section CallTarget+Inlining+NGEN
This PR (8127) - mean (894ms) : 856, 932
master - mean (892ms) : 861, 922
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
...adog.Trace.ClrProfiler.Managed.Tests/AutoInstrumentation/MongoDb/TruncatedTextWriterTests.cs
Show resolved
Hide resolved
7a60ba9 to
9215be8
Compare
BenchmarksBenchmark execution time: 2026-02-04 09:56:16 Comparing candidate commit aceccd9 in PR branch Found 8 performance improvements and 10 performance regressions! Performance is the same for 157 metrics, 17 unstable metrics. scenario:Benchmarks.Trace.AgentWriterBenchmark.WriteAndFlushEnrichedTraces net472
scenario:Benchmarks.Trace.AgentWriterBenchmark.WriteAndFlushEnrichedTraces net6.0
scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleMoreComplexBody net6.0
scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleSimpleBody net6.0
scenario:Benchmarks.Trace.Asm.AppSecEncoderBenchmark.EncodeLegacyArgs netcoreapp3.1
scenario:Benchmarks.Trace.Asm.AppSecWafBenchmark.RunWafRealisticBenchmark netcoreapp3.1
scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces net472
scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces netcoreapp3.1
scenario:Benchmarks.Trace.CharSliceBenchmark.OptimizedCharSliceWithPool net6.0
scenario:Benchmarks.Trace.GraphQLBenchmark.ExecuteAsync net472
scenario:Benchmarks.Trace.ILoggerBenchmark.EnrichedLog net6.0
scenario:Benchmarks.Trace.ILoggerBenchmark.EnrichedLog netcoreapp3.1
scenario:Benchmarks.Trace.Log4netBenchmark.EnrichedLog netcoreapp3.1
scenario:Benchmarks.Trace.SingleSpanAspNetCoreBenchmark.SingleSpanAspNetCore net6.0
scenario:Benchmarks.Trace.SingleSpanAspNetCoreBenchmark.SingleSpanAspNetCore netcoreapp3.1
scenario:Benchmarks.Trace.SpanBenchmark.StartFinishTwoScopes net6.0
|
45532b0 to
aceccd9
Compare
Summary of changes
Fixing a test that was failing on my env, because of a
,instead of a..Reason for change
The test was the issue not the code it tested.
Implementation details
Just adding a good old
CultureInfo.InvariantCultureas Lucas has been doing everywhere since ever :pTest coverage
N/A