[Test Package Versions Bump] Updating package versions - #8955
[Test Package Versions Bump] Updating package versions#8955dd-octo-sts[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 988773d47e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "maxVersionSupportedInclusive": "4.0.102", | ||
| "maxVersionAvailableInclusive": "4.0.102", | ||
| "maxVersionTestedInclusive": "4.0.101.4" | ||
| "maxVersionSupportedInclusive": "4.0.102.1", |
There was a problem hiding this comment.
Do not publish cooled-down packages as supported
For package bumps that hit the 2-day cooldown, this file now advances maxVersionSupportedInclusive to the ignored NuGet version even though the generated test matrices still stop at the kept version (for this entry, PackageVersionsLatestMajors.g.cs still uses 4.0.102, not 4.0.102.1). That makes consumers of supported_versions.json report the cooled-down version as supported before it has been included in the package-version tests/manual review; the same pattern appears for the other cooldown entries in this commit. Please leave the supported cap at the kept/tested version until the cooldown passes.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Interesting, this is wrong @bouwkast 🤔 Looks like it happens when we have both a "valid" bump and a cooldown bump
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8955) 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 (8955) - mean (72ms) : 69, 75
master - mean (71ms) : 69, 73
section Bailout
This PR (8955) - mean (78ms) : 75, 82
master - mean (75ms) : 73, 77
section CallTarget+Inlining+NGEN
This PR (8955) - mean (1,091ms) : 1041, 1140
master - mean (1,097ms) : 1040, 1155
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 (8955) - mean (113ms) : 107, 119
master - mean (110ms) : 108, 113
section Bailout
This PR (8955) - mean (113ms) : 108, 118
master - mean (111ms) : 109, 113
section CallTarget+Inlining+NGEN
This PR (8955) - mean (783ms) : 763, 802
master - mean (790ms) : 767, 814
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8955) - mean (101ms) : 94, 108
master - mean (98ms) : 96, 101
section Bailout
This PR (8955) - mean (103ms) : 97, 108
master - mean (103ms) : 98, 108
section CallTarget+Inlining+NGEN
This PR (8955) - mean (952ms) : 914, 990
master - mean (946ms) : 902, 989
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8955) - mean (96ms) : 93, 99
master - mean (96ms) : 92, 100
section Bailout
This PR (8955) - mean (101ms) : 96, 106
master - mean (97ms) : 95, 99
section CallTarget+Inlining+NGEN
This PR (8955) - mean (818ms) : 783, 854
master - mean (820ms) : 785, 855
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 (8955) - mean (192ms) : 188, 196
master - mean (191ms) : 188, 194
section Bailout
This PR (8955) - mean (195ms) : 194, 197
master - mean (195ms) : 193, 197
section CallTarget+Inlining+NGEN
This PR (8955) - mean (1,175ms) : 1129, 1221
master - mean (1,171ms) : 1125, 1217
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 (8955) - mean (274ms) : 270, 277
master - mean (273ms) : 269, 277
section Bailout
This PR (8955) - mean (273ms) : 271, 276
master - mean (273ms) : 270, 277
section CallTarget+Inlining+NGEN
This PR (8955) - mean (939ms) : 922, 957
master - mean (935ms) : 912, 958
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8955) - mean (268ms) : 263, 273
master - mean (268ms) : 264, 272
section Bailout
This PR (8955) - mean (267ms) : 264, 270
master - mean (267ms) : 265, 270
section CallTarget+Inlining+NGEN
This PR (8955) - mean (1,137ms) : 1076, 1199
master - mean (1,143ms) : 1096, 1190
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8955) - mean (265ms) : 262, 269
master - mean (265ms) : 261, 269
section Bailout
This PR (8955) - mean (266ms) : 263, 268
master - mean (265ms) : 263, 268
section CallTarget+Inlining+NGEN
This PR (8955) - mean (1,026ms) : 992, 1060
master - mean (1,028ms) : 983, 1073
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
BenchmarksBenchmark execution time: 2026-08-03 13:23:34 Comparing candidate commit 294eaec in PR branch Found 0 performance improvements and 2 performance regressions! Performance is the same for 70 metrics, 0 unstable metrics, 68 known flaky benchmarks, 58 flaky benchmarks without significant changes.
|
988773d to
38e845b
Compare
38e845b to
294eaec
Compare
Updates the package versions for integration tests. See below for a summary of the changes.
Package Version Updates
New Major Versions Available
These have a new major outside our supported range. Review whether to widen the
integration's
MaximumVersionand the definition'sMaxVersionExclusive/SpecificVersions,or add a new split entry, to adopt.
Package Version Cooldown Report
The following versions were published less than 2 days ago and have been ignored.
These require manual review before inclusion.