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

improve logging #110

Closed OliverMKing closed 9 months ago

OliverMKing commented 9 months ago

Description

Improves logging in our controllers

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

E2e and unit

Checklist:

coveralls commented 9 months ago

Pull Request Test Coverage Report for Build 6483628626


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/controller/osm/ingress_cert_config_reconciler.go 9 10 90.0%
pkg/controller/keyvault/placeholder_pod.go 11 13 84.62%
pkg/controller/ingress/concurrency_watchdog.go 15 18 83.33%
pkg/controller/keyvault/event_mirror.go 23 28 82.14%
<!-- Total: 80 91 87.91% -->
Files with Coverage Reduction New Missed Lines %
pkg/controller/keyvault/event_mirror.go 1 88.89%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 6434739717: 0.1%
Covered Lines: 2029
Relevant Lines: 2284

💛 - Coveralls
OliverMKing commented 9 months ago

/ok-to-test sha=996e000