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

switch all controllers to use crd as source of truth #128

Closed OliverMKing closed 8 months ago

OliverMKing commented 8 months ago

Description

Switches the crd to be the source of truth for all controllers.

TODO next pr: push event to crd if fail to reconcile

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 6801242937


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/controller/keyvault/placeholder_pod.go 2 4 50.0%
pkg/controller/ingress/concurrency_watchdog.go 32 35 91.43%
pkg/controller/nginxingress/nginx_ingress_controller.go 23 28 82.14%
pkg/controller/nginxingress/index.go 43 60 71.67%
pkg/controller/controller.go 0 44 0.0%
<!-- Total: 139 210 66.19% -->
Files with Coverage Reduction New Missed Lines %
pkg/controller/nginxingress/nginx_ingress_controller.go 1 68.59%
pkg/controller/controller.go 2 17.22%
<!-- Total: 3 -->
Totals Coverage Status
Change from base Build 6758002808: -0.04%
Covered Lines: 2763
Relevant Lines: 3588

💛 - Coveralls