-
Notifications
You must be signed in to change notification settings - Fork 147
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
[IAST] Skip failing tests #6455
[IAST] Skip failing tests #6455
Conversation
@@ -573,6 +573,8 @@ await VerifyHelper.VerifySpans(spansFiltered, settings) | |||
[Trait("RunOnWindows", "True")] | |||
public async Task TestIastStoredXssRequest() | |||
{ | |||
throw new SkipException("End point missing due to changes in Samples.Security.AspNetCore2 - refactoring in progess to restore"); |
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.
This is the same as doing this, right?
[SkippableFact(Skip = "End point missing due to changes in Samples.Security.AspNetCore2 - refactoring in progess to restore")]
We always want to skip, right? These tests only run on .NET Core 2.1?
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.
Yes, definitely only run on Core 2.1, we always want to skip because it can't work anywhere. I wasn't aware of the other way to skip test, I'll use that next time.
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.
Should be re-enabled soon anyway.
Datadog ReportBranch report: ❌ 3 Failed (0 Known Flaky), 464051 Passed, 2859 Skipped, 20h 9m 59.94s Total Time ❌ Failed Tests (3)
|
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 (6455) - mean (68ms) : 66, 71
. : milestone, 68,
master - mean (68ms) : 65, 71
. : milestone, 68,
section CallTarget+Inlining+NGEN
This PR (6455) - mean (982ms) : 957, 1008
. : milestone, 982,
master - mean (980ms) : 952, 1008
. : milestone, 980,
gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6455) - mean (108ms) : 106, 110
. : milestone, 108,
master - mean (107ms) : 105, 109
. : milestone, 107,
section CallTarget+Inlining+NGEN
This PR (6455) - mean (676ms) : 663, 690
. : milestone, 676,
master - mean (679ms) : 663, 695
. : milestone, 679,
gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6455) - mean (91ms) : 89, 93
. : milestone, 91,
master - mean (91ms) : 89, 93
. : milestone, 91,
section CallTarget+Inlining+NGEN
This PR (6455) - mean (632ms) : 615, 649
. : milestone, 632,
master - mean (633ms) : 616, 650
. : milestone, 633,
gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6455) - mean (194ms) : 190, 198
. : milestone, 194,
master - mean (193ms) : 188, 198
. : milestone, 193,
section CallTarget+Inlining+NGEN
This PR (6455) - mean (1,105ms) : 1077, 1132
. : milestone, 1105,
master - mean (1,098ms) : 1072, 1123
. : milestone, 1098,
gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6455) - mean (278ms) : 274, 282
. : milestone, 278,
master - mean (277ms) : 273, 281
. : milestone, 277,
section CallTarget+Inlining+NGEN
This PR (6455) - mean (867ms) : 842, 892
. : milestone, 867,
master - mean (868ms) : 840, 895
. : milestone, 868,
gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (6455) - mean (267ms) : 262, 272
. : milestone, 267,
master - mean (266ms) : 262, 271
. : milestone, 266,
section CallTarget+Inlining+NGEN
This PR (6455) - mean (844ms) : 805, 884
. : milestone, 844,
master - mean (853ms) : 824, 881
. : milestone, 853,
|
Throughput/Crank Report ⚡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 (6455) (11.244M) : 0, 11243701
master (11.163M) : 0, 11163379
benchmarks/2.9.0 (11.033M) : 0, 11032866
section Automatic
This PR (6455) (7.312M) : 0, 7312276
master (7.265M) : 0, 7264817
benchmarks/2.9.0 (7.786M) : 0, 7785853
section Trace stats
master (7.670M) : 0, 7670252
section Manual
master (11.116M) : 0, 11115719
section Manual + Automatic
This PR (6455) (6.752M) : 0, 6751641
master (6.789M) : 0, 6789402
section DD_TRACE_ENABLED=0
master (10.411M) : 0, 10410923
gantt
title Throughput Linux arm64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (6455) (9.635M) : 0, 9635072
master (9.330M) : 0, 9329645
benchmarks/2.9.0 (9.495M) : 0, 9494821
section Automatic
This PR (6455) (6.487M) : 0, 6486904
master (6.258M) : 0, 6257542
section Trace stats
master (6.725M) : 0, 6724804
section Manual
master (9.564M) : 0, 9564271
section Manual + Automatic
This PR (6455) (5.971M) : 0, 5970998
master (5.828M) : 0, 5828361
section DD_TRACE_ENABLED=0
master (8.501M) : 0, 8500849
gantt
title Throughput Windows x64 (Total requests)
dateFormat X
axisFormat %s
section Baseline
This PR (6455) (10.210M) : 0, 10210091
master (9.811M) : 0, 9810785
benchmarks/2.9.0 (10.020M) : 0, 10019592
section Automatic
This PR (6455) (6.299M) : 0, 6299034
master (6.389M) : 0, 6389247
benchmarks/2.9.0 (7.255M) : 0, 7255257
section Trace stats
master (6.930M) : 0, 6930451
section Manual
master (9.742M) : 0, 9741749
section Manual + Automatic
This PR (6455) (5.790M) : 0, 5789682
master (5.896M) : 0, 5896103
section DD_TRACE_ENABLED=0
master (9.127M) : 0, 9127235
|
Benchmarks Report for tracer 🐌Benchmarks for #6455 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 - Slower
|
Benchmark | diff/base | Base Median (ns) | Diff Median (ns) | Modality |
---|---|---|---|---|
Benchmarks.Trace.SpanBenchmark.StartFinishSpan‑net6.0 | 1.155 | 407.64 | 470.66 |
Raw results
Branch | Method | Toolchain | Mean | StdError | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
---|---|---|---|---|---|---|---|---|---|
master | StartFinishSpan |
net6.0 | 408ns | 0.185ns | 0.716ns | 0.00801 | 0 | 0 | 576 B |
master | StartFinishSpan |
netcoreapp3.1 | 571ns | 0.368ns | 1.38ns | 0.0077 | 0 | 0 | 576 B |
master | StartFinishSpan |
net472 | 715ns | 0.553ns | 2.14ns | 0.0918 | 0 | 0 | 578 B |
master | StartFinishScope |
net6.0 | 487ns | 0.264ns | 0.989ns | 0.00973 | 0 | 0 | 696 B |
master | StartFinishScope |
netcoreapp3.1 | 732ns | 0.406ns | 1.57ns | 0.00926 | 0 | 0 | 696 B |
master | StartFinishScope |
net472 | 901ns | 0.409ns | 1.58ns | 0.104 | 0 | 0 | 658 B |
#6455 | StartFinishSpan |
net6.0 | 471ns | 0.287ns | 1.11ns | 0.00803 | 0 | 0 | 576 B |
#6455 | StartFinishSpan |
netcoreapp3.1 | 560ns | 0.472ns | 1.83ns | 0.00764 | 0 | 0 | 576 B |
#6455 | StartFinishSpan |
net472 | 665ns | 0.607ns | 2.35ns | 0.0917 | 0 | 0 | 578 B |
#6455 | StartFinishScope |
net6.0 | 480ns | 0.57ns | 2.21ns | 0.00983 | 0 | 0 | 696 B |
#6455 | StartFinishScope |
netcoreapp3.1 | 728ns | 0.513ns | 1.99ns | 0.00945 | 0 | 0 | 696 B |
#6455 | StartFinishScope |
net472 | 920ns | 0.527ns | 2.04ns | 0.104 | 0 | 0 | 658 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 | 644ns | 0.585ns | 2.27ns | 0.00991 | 0 | 0 | 696 B |
master | RunOnMethodBegin |
netcoreapp3.1 | 970ns | 1.02ns | 3.96ns | 0.00928 | 0 | 0 | 696 B |
master | RunOnMethodBegin |
net472 | 1.11μs | 0.687ns | 2.66ns | 0.104 | 0 | 0 | 658 B |
#6455 | RunOnMethodBegin |
net6.0 | 590ns | 0.303ns | 1.13ns | 0.00977 | 0 | 0 | 696 B |
#6455 | RunOnMethodBegin |
netcoreapp3.1 | 975ns | 0.849ns | 3.29ns | 0.00923 | 0 | 0 | 696 B |
#6455 | RunOnMethodBegin |
net472 | 1.11μs | 0.466ns | 1.74ns | 0.104 | 0 | 0 | 658 B |
Summary of changes
Skip two tests that now fail on master. They fail because the endpoint in the test app they use was removed.
This wasn't spotted in the PR because they are only run on master.
A larger refactoring to fix this is in progress.