Update default Azure VM SKU to Standard_D2ds_v5#5996
Update default Azure VM SKU to Standard_D2ds_v5#5996mboersma wants to merge 1 commit intokubernetes-sigs:mainfrom
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/test pull-cluster-api-provider-azure-e2e |
364856c to
4bb058e
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5996 +/- ##
=======================================
Coverage 44.40% 44.41%
=======================================
Files 280 280
Lines 25377 25377
=======================================
+ Hits 11269 11271 +2
+ Misses 13295 13293 -2
Partials 813 813 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/test pull-cluster-api-provider-azure-e2e-optional |
|
/retitle [WIP] Update default Azure VM SKUs used in testing and templates AKS tests failed with: |
62c6c2c to
3c2a139
Compare
|
/hold In conjunction with this, we should ensure there is increased quota for this SKU family in expected test regions. I'll do that if this passes. |
|
/retest |
Use Standard_D2ds_v5 and Standard_D4s_v5
3c2a139 to
26353b5
Compare
|
@mboersma: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
jackfrancis
left a comment
There was a problem hiding this comment.
We don't have quota for the v5 SKU, so let's table this.
/close
|
@jackfrancis: Closed this PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Follows up on #6017 to update remaining default Azure VM SKUs used in testing and templates since the current choices are now deprecated.
Overall:
Standard_D2s_v3->Standard_D2ds_v5.Standard_D4s_v3->Standard_D4s_v5.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
See the migration guide for more information. The current SKUs will be phased out in 2028, so this isn't urgent.
TODOs:
Release note: