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

add crd webhooks #123

Closed OliverMKing closed 8 months ago

OliverMKing commented 8 months ago

Description

Adds crd webhooks.

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Tested locally and unit tested. Will be e2e tested in a future PR.

Checklist:

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6737124812


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/webhook/webhook.go 85 127 66.93%
pkg/webhook/cert.go 24 86 27.91%
pkg/webhook/nginxingress.go 96 177 54.24%
pkg/controller/controller.go 0 86 0.0%
<!-- Total: 224 495 45.25% -->
Files with Coverage Reduction New Missed Lines %
pkg/controller/controller.go 5 20.34%
<!-- Total: 5 -->
Totals Coverage Status
Change from base Build 6722886340: -5.6%
Covered Lines: 2355
Relevant Lines: 2999

💛 - Coveralls
OliverMKing commented 8 months ago

/ok-to-test sha=399b4f

OliverMKing commented 8 months ago

/ok-to-test sha=399b54f