Azure / aks-app-routing-operator

Kubernetes operator that implements AKS Application Routing
https://learn.microsoft.com/en-us/azure/aks/app-routing
MIT License
29 stars 20 forks source link

split off webhook certs management into separate command #131

Closed OliverMKing closed 8 months ago

OliverMKing commented 8 months ago

Description

splits our webhook certs management code into a separate command so it can be more flexible.

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Tested locally and unit tested

Checklist:

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6832810978


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/webhook/webhook.go 21 23 91.3%
pkg/controller/controller.go 0 10 0.0%
pkg/webhook/cert.go 103 131 78.63%
<!-- Total: 150 190 78.95% -->
Files with Coverage Reduction New Missed Lines %
pkg/webhook/webhook.go 1 86.89%
pkg/webhook/cert.go 6 75.52%
<!-- Total: 7 -->
Totals Coverage Status
Change from base Build 6813150888: 2.0%
Covered Lines: 2895
Relevant Lines: 3665

💛 - Coveralls