search

Azure / karpenter-provider-azure

AKS Karpenter Provider
Apache License 2.0
308 stars 46 forks source link

fix: specify location when creating ACR #414

Closed tallaxes closed 1 week ago

tallaxes commented 2 weeks ago

Description

Explicitly specifying the location for ACR seems to be fixing "ACR not found" issues during E2E. (Would like to follow-up to see why we are trying to create ACR twice ...)

It also switches the default region from eastus (which appears to have some capacity constraints right now) to southcentralus. We don't have GPU quota there, so GPU tests will fail for now.

How was this change tested?

Does this change impact docs?

Release Note

coveralls commented 2 weeks ago

Pull Request Test Coverage Report for Build 9590195851

Details

Totals Coverage Status
Change from base Build 9587074007: 0.0%
Covered Lines: 36286
Relevant Lines: 37115
💛 - Coveralls
coveralls commented 2 weeks ago

Pull Request Test Coverage Report for Build 9590419539

Details

Totals Coverage Status
Change from base Build 9587074007: 0.0%
Covered Lines: 36286
Relevant Lines: 37115
💛 - Coveralls
coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9619131176

Details

Totals Coverage Status
Change from base Build 9603102120: 0.0%
Covered Lines: 36287
Relevant Lines: 37117
💛 - Coveralls
coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9621866101

Details

Totals Coverage Status
Change from base Build 9621183275: 0.006%
Covered Lines: 36287
Relevant Lines: 37117
💛 - Coveralls