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

added owner reference to webhook config #129

Closed bosesuneha closed 8 months ago

bosesuneha commented 8 months ago

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. Added owner reference to bind webhooks to the app-routing-system namespace. Fixes # (issue) Feature # (details)

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 6813147388

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/controller/controller.go 0 1 0.0%
pkg/webhook/webhook.go 7 9 77.78%
<!-- Total: 10 13 76.92% -->
Totals Coverage Status
Change from base Build 6813143077: -0.01%
Covered Lines: 2779
Relevant Lines: 3611

💛 - Coveralls