[Test Package Versions Bump] Updating package versions#8502
[Test Package Versions Bump] Updating package versions#8502andrewlock merged 1 commit intomasterfrom
Conversation
andrewlock
left a comment
There was a problem hiding this comment.
LGTM, but we shouldn't merge these before #8478, otherwise we'll get conflicts
68f5432 to
ba23446
Compare
BenchmarksBenchmark execution time: 2026-04-24 11:36:24 Comparing candidate commit b206284 in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 26 metrics, 0 unstable metrics, 60 known flaky benchmarks, 27 flaky benchmarks without significant changes.
|
ba23446 to
253363b
Compare
## Summary of changes This removes the dependabot configuration and related code/projects for updating instrumentations. ## Reason for change Dependabot would trigger and then open a PR like so [Bump log4net from 3.3.0 to 3.3.1](#8501). Then our workflow would trigger and raise this PR [[Test Package Versions Bump] Updating package versions](#8502) Ultimately we don't really act on these dependabot PRs so they don't add a ton of value. To compensate for the potential loss in visibility I've opted to add a single additional run on Wednesday which will regenerate the `[Test Package Versions Bump]` if still open from Sunday's. Depending on how this goes we may want to remove the Wednesday additional run, but running it on Wednesday will mean that any cooldown packages skipped on Sunday will get bumped which may be helpful. ## Implementation details - Removed all the dependabot honeypot projects - I don't think we need them anymore they were used by dependabot to do the bumping - Removed all code that updated/touched those projects - Removed the dependabot config for the integrations - Updated/Removed the github workflow that opens the `[Test Package Versions Bump]` to no longer look for the dependabot PRs (will need to close them manually) and to run on Wednesday too. ## Test coverage none ## Other details <!-- Fixes #{issue} --> <!--⚠️ Note: Where possible, please obtain 2 approvals prior to merging. Unless CODEOWNERS specifies otherwise, for external teams it is typically best to have one review from a team member, and one review from apm-dotnet. Trivial changes do not require 2 reviews. MergeQueue is NOT enabled in this repository. If you have write access to the repo, the PR has 1-2 approvals (see above), and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #apm-dotnet channel in Slack. --> --------- Co-authored-by: Andrew Lock <andrew.lock@datadoghq.com>
253363b to
b206284
Compare
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8502) 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 (8502) - mean (73ms) : 70, 76
master - mean (73ms) : 70, 76
section Bailout
This PR (8502) - mean (77ms) : 74, 79
master - mean (80ms) : 76, 83
section CallTarget+Inlining+NGEN
This PR (8502) - mean (1,079ms) : 1037, 1121
master - mean (1,075ms) : 1028, 1122
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 (8502) - mean (114ms) : 109, 119
master - mean (116ms) : 110, 121
section Bailout
This PR (8502) - mean (115ms) : 112, 118
master - mean (117ms) : 111, 122
section CallTarget+Inlining+NGEN
This PR (8502) - mean (783ms) : 753, 812
master - mean (776ms) : 745, 807
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8502) - mean (102ms) : 98, 105
master - mean (102ms) : 96, 109
section Bailout
This PR (8502) - mean (102ms) : 100, 104
master - mean (105ms) : 99, 111
section CallTarget+Inlining+NGEN
This PR (8502) - mean (941ms) : 905, 976
master - mean (939ms) : 898, 980
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8502) - mean (105ms) : 99, 110
master - mean (101ms) : 96, 105
section Bailout
This PR (8502) - mean (102ms) : 98, 107
master - mean (102ms) : 100, 104
section CallTarget+Inlining+NGEN
This PR (8502) - mean (825ms) : 792, 858
master - mean (824ms) : 782, 866
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 (8502) - mean (203ms) : 195, 212
master - mean (201ms) : 196, 207
section Bailout
This PR (8502) - mean (206ms) : 200, 212
master - mean (205ms) : 202, 208
section CallTarget+Inlining+NGEN
This PR (8502) - mean (1,198ms) : 1144, 1251
master - mean (1,195ms) : 1130, 1261
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 (8502) - mean (291ms) : 279, 302
master - mean (289ms) : 280, 298
section Bailout
This PR (8502) - mean (293ms) : 280, 305
master - mean (290ms) : 281, 299
section CallTarget+Inlining+NGEN
This PR (8502) - mean (957ms) : 928, 985
master - mean (949ms) : 918, 980
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8502) - mean (285ms) : 274, 295
master - mean (284ms) : 273, 294
section Bailout
This PR (8502) - mean (286ms) : 274, 298
master - mean (283ms) : 275, 290
section CallTarget+Inlining+NGEN
This PR (8502) - mean (1,158ms) : 1125, 1191
master - mean (1,156ms) : 1119, 1193
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8502) - mean (282ms) : 270, 293
master - mean (284ms) : 273, 296
section Bailout
This PR (8502) - mean (283ms) : 271, 295
master - mean (287ms) : 275, 300
section CallTarget+Inlining+NGEN
This PR (8502) - mean (1,042ms) : 994, 1089
master - mean (1,064ms) : 963, 1165
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Updates the package versions for integration tests.
Package Version Cooldown Report
The following versions were published less than 2 days ago and have been overridden.
These require manual review before inclusion.