Skip to content
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

test(e2e): switch to westus3 for e2e #440

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

charliedmcb
Copy link
Collaborator

Fixes #

Description
We've been experiencing some allocation issues within the current testing regions.

Did some testing to find a new region, which westus3 seems to be rather well suited for our needs, so switching the testing pipeline there.

How was this change tested?
Ran the E2E pipeline for westus3:
https://github.com/Azure/karpenter-provider-azure/actions/runs/9964029669/job/27531436071

While not all successful, did a sanity check within the subscription on quota, and public doc support for multi-zone region, and this appears to be a good region change:
https://learn.microsoft.com/en-us/azure/reliability/availability-zones-service-support
*

Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: #
  • No

Release Note

NONE

@coveralls
Copy link

Pull Request Test Coverage Report for Build 9964502625

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 97.706%

Totals Coverage Status
Change from base Build 9912418452: 0.005%
Covered Lines: 36287
Relevant Lines: 37139

💛 - Coveralls

@tallaxes tallaxes added the area/ci Issues or PRs related to ci label Jul 16, 2024
@charliedmcb charliedmcb merged commit 6970164 into main Jul 16, 2024
11 checks passed
@charliedmcb charliedmcb deleted the charliedmcb/switchE2EToWestus3 branch July 16, 2024 21:58
Bryce-Soghigian pushed a commit that referenced this pull request Sep 12, 2024
Co-authored-by: Charlie McBride <Charlie.McBride@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues or PRs related to ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants