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

Adding webhook to prom metrics #125

Closed bfoley13 closed 8 months ago

bfoley13 commented 8 months ago

Description

This changes adds mutation/validation webhooks into our current Prometheus metrics.

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a breaking change which will impact consuming tool(s)?

Checklist:

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6749516416

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report. To ensure accuracy in future PRs, please see these guidelines. A quick fix for this PR: rebase it; your next report should be accurate.


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/webhook/nginxingress.go 22 27 81.48%
<!-- Total: 22 27 81.48% -->
Totals Coverage Status
Change from base Build 6740427943: 0.02%
Covered Lines: 2369
Relevant Lines: 3016

💛 - Coveralls