Closed Bryce-Soghigian closed 1 month ago
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
pkg/providers/imagefamily/bootstrap/aksbootstrap.go | 17 | 85.53% | ||
<!-- | Total: | 17 | --> |
Totals | |
---|---|
Change from base Build 9074172591: | -0.008% |
Covered Lines: | 36286 |
Relevant Lines: | 37115 |
I would recommend running it against the failing case (e.g., undoing https://github.com/Azure/karpenter-provider-azure/pull/220) as well to validate it.
gonna rewrite this on the azure repo rather than my fork.
Fixes #
Description As part of the efforts to fix 1.30 bootstrap we removed a flag that removes functionality for out of tree providers. One of the side affects of this is that ACR Pull will not work in the same way. We needed a way to validate this image pull works in a reproducible and consistent way.
This test suite is that. Currently it has to be ran locally to target specific k8s versions so a nice followup at lower priority would be to add the ability to run our e2es against a specific k8s version. How was this change tested?
Does this change impact docs?
Release Note