Replace ingress-nginx with Traefik as recommended ingress controller#413
Replace ingress-nginx with Traefik as recommended ingress controller#413welteki wants to merge 1 commit intoopenfaas:masterfrom
Conversation
This comment has been minimized.
This comment has been minimized.
0ba02b1 to
740b89c
Compare
This comment has been minimized.
This comment has been minimized.
I feel like we should switch completely to fully formed helm commands and maintain this ourselves. An AI agent could probably do this for you quickly. |
740b89c to
4ef2e49
Compare
This comment has been minimized.
This comment has been minimized.
4ef2e49 to
9e1a4d9
Compare
This comment has been minimized.
This comment has been minimized.
9e1a4d9 to
8666055
Compare
This comment has been minimized.
This comment has been minimized.
Update documentation across multiple pages to recommend Traefik as the default ingress controller instead of ingress-nginx: - Switch ingress controller references from nginx to Traefik - Update installation commands to use arkade install traefik2 - Replace nginx-specific annotations with Traefik equivalents - Update ingressClassName from nginx to traefik - Add Traefik timeout configuration guide Signed-off-by: Han Verstraete (OpenFaaS Ltd) <han@openfaas.com>
AI Pull Request OverviewSummary
Approval rating (1-10)9/10 - Thorough update replacing retired ingress controller with recommended Traefik, ensuring documentation accuracy and future-proofing user setups. Summary per fileSummary per file
Overall AssessmentThe pull request comprehensively updates the OpenFaaS documentation to replace references to the retiring ingress-nginx project with Traefik as the recommended ingress controller. This change is timely given ingress-nginx's announced retirement in March 2026 and ensures users are directed toward supported, actively maintained software. The updates maintain consistency across documentation files and provide clear, actionable guidance for Traefik configuration including timeouts. No technical issues or risks identified in the Kubernetes configurations presented. Detailed ReviewDetailed Reviewdocs/reference/tls-functions.md
docs/reference/tls-openfaas.md
docs/tutorials/expanded-timeouts.md
docs/deployment/kubernetes.md
Risk Assessment: No security, performance, or compatibility risks identified. The changes are purely documentation updates that improve accuracy without introducing breaking changes for existing deployments. Testing Considerations: While documentation changes don't require runtime testing, the configurations provided should be validated against current Traefik and Kubernetes versions to ensure continued accuracy. AI agent details. |
Description
Replace all ingress-nginx references with Traefik across the documentation:
arkade install traefik2ingressClassNamefromnginxtotraefikMotivation and Context
The Ingress Nginx project will be retired in March 2026. This change updates the documentation to recommend Traefik as the default ingress controller going forward.
How Has This Been Tested?
Documentation changes reviewed for correctness of Traefik configuration, service names, and command references.
Testing of updated examples is in progress.
Types of changes
Checklist:
git commit -s