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 new events on published to Ingress in case of reconcile failures #130

Closed OliverMKing closed 8 months ago

OliverMKing commented 8 months ago

Description

Add some new events that are published to cx Ingresses utilizing App Routing when we fail to upsert resources needed for the Ingress to function. Provides significantly increased customer visibility.

We publish to the Ingress object instead of the NginxIngressController crd because

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Tested locally

Checklist:

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6804380187


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/controller/keyvault/placeholder_pod.go 6 7 85.71%
pkg/controller/keyvault/ingress_secret_provider_class.go 1 3 33.33%
pkg/controller/osm/ingress_backend_reconciler.go 7 9 77.78%
<!-- Total: 14 19 73.68% -->
Totals Coverage Status
Change from base Build 6803409995: -0.04%
Covered Lines: 2774
Relevant Lines: 3604

💛 - Coveralls