Azure / karpenter-provider-azure

AKS Karpenter Provider
Apache License 2.0
394 stars 65 forks source link

Make webhooks work in AKS CCP context #537

Closed matthchr closed 1 month ago

matthchr commented 1 month ago

Description This requires quite a bit of hacking, mostly overriding certain things in the ctx. The major items are:

How was this change tested?

Does this change impact docs?

Release Note

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 11485021484

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/alt/knative/pkg/webhook/resourcesemantics/conversion/reconciler.go 49 52 94.23%
pkg/alt/knative/pkg/webhook/resourcesemantics/conversion/conversion.go 131 135 97.04%
pkg/alt/knative/pkg/webhook/resourcesemantics/conversion/controller.go 50 55 90.91%
pkg/alt/knative/pkg/webhook/resourcesemantics/conversion/internal/zz_generated.deepcopy.go 62 74 83.78%
pkg/alt/knative/pkg/webhook/resourcesemantics/conversion/internal/types.go 92 109 84.4%
<!-- Total: 396 437 90.62% -->
Totals Coverage Status
Change from base Build 11454836497: -0.08%
Covered Lines: 36961
Relevant Lines: 37883

💛 - Coveralls