Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(ingress-nginx): remove need for individual ingress annotations #8446

Conversation

onedr0p
Copy link
Owner

@onedr0p onedr0p commented Dec 10, 2024

Less annotations, more wow

@onedr0p onedr0p changed the title STEVEN Revert "Revert "feat(ingress-nginx): remove need for individual ingress annotations"" feat(ingress-nginx): remove need for individual ingress annotations Dec 10, 2024
@bot-ross bot-ross bot added area/kubernetes Changes made in the kubernetes directory cluster/main area/docs labels Dec 10, 2024
@onedr0p onedr0p merged commit ccb59e0 into main Dec 10, 2024
4 checks passed
@onedr0p onedr0p deleted the revert-8445-revert-8444-revert-8443-revert-8441-no-ingress-annotations branch December 10, 2024 00:44
@alex-matthews
Copy link

Worth reflecting in cluster-template?

@onedr0p
Copy link
Owner Author

onedr0p commented Dec 21, 2024

I thought about it but because it only uses external-dns for cloudflare and k8s-gateway for internal DNS it might not work. If you run this exact setup still feel free to open a PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/docs area/kubernetes Changes made in the kubernetes directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants