Skip to content

Traefik ingress#29

Merged
eleonov merged 18 commits intomainfrom
traefik-ingress
Dec 23, 2025
Merged

Traefik ingress#29
eleonov merged 18 commits intomainfrom
traefik-ingress

Conversation

@akiyashkin-ronasit
Copy link
Contributor

Changes

  • Migrated Laravel chart from Nginx Ingress to Traefik
  • Added Traefik middleware support:
    • Forwarded headers middleware (X-Forwarded-Proto, X-Forwarded-Port, X-Forwarded-Host)
    • Request/response buffering middleware (configurable max sizes)
    • IP allowlist middleware (optional IP whitelisting)
    • Noindex middleware (optional X-Robots-Tag header)
  • Updated values.yaml ingress configuration:
    • Simplified hosts configuration (removed path-based routing)
    • Added Traefik-specific settings (entryPoints, certResolver)
    • Added middleware configuration options
    • Removed deprecated Nginx annotations
  • Bumped chart version to 2.8.0
  • Updated dependencies (PostgreSQL 16.7.27, Redis 20.13.4)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the Laravel Helm chart from Nginx Ingress Controller to Traefik, representing a significant change in how ingress is configured and managed. The migration introduces Traefik-specific Custom Resource Definitions (IngressRoute and Middleware) to replace the standard Kubernetes Ingress resource, providing more granular control over request handling through dedicated middleware components.

Key Changes:

  • Complete replacement of standard Kubernetes Ingress with Traefik IngressRoute and Middleware CRDs
  • Introduction of four configurable middleware components: forwarded headers, request/response buffering, IP allowlisting, and noindex headers
  • Simplified ingress configuration structure with Traefik-specific settings (entryPoints, certResolver)

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 11 comments.

Show a summary per file
File Description
charts/laravel/values.yaml Removed Nginx ingress configuration and annotations; added Traefik-specific configuration with middleware options
charts/laravel/templates/ingress.yaml Deleted standard Kubernetes Ingress template (entire file removed)
charts/laravel/templates/ingress-route.yaml Created new Traefik IngressRoute resource with conditional middleware references
charts/laravel/templates/ingress-middleware-headers.yaml Created middleware for X-Forwarded-* headers management
charts/laravel/templates/ingress-middleware-noindex.yaml Created optional middleware for X-Robots-Tag header
charts/laravel/templates/ingress-middleware-buffering.yaml Created optional middleware for request/response buffering with configurable limits
charts/laravel/templates/ingress-ip-allowlist-middleware.yaml Created optional middleware for IP-based access control
charts/laravel/templates/NOTES.txt Updated variable names to match new ingress configuration structure
charts/laravel/README.md Updated version badge and ingress-related documentation entries
charts/laravel/Chart.yaml Bumped chart version to 2.8.0
charts/laravel/Chart.lock Updated dependency versions (PostgreSQL 16.7.27, Redis 20.13.4) and regenerated digest

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

akiyashkin-ronasit and others added 6 commits December 11, 2025 08:43
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…yaml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI commented Dec 18, 2025

@akiyashkin-ronasit I've opened a new pull request, #30, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Contributor

Copilot AI commented Dec 18, 2025

@akiyashkin-ronasit I've opened a new pull request, #31, to work on those changes. Once the pull request is ready, I'll request review from you.

akiyashkin-ronasit and others added 5 commits December 18, 2025 09:45
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…values.yaml

Co-authored-by: akiyashkin-ronasit <162979216+akiyashkin-ronasit@users.noreply.github.com>
Co-authored-by: akiyashkin-ronasit <162979216+akiyashkin-ronasit@users.noreply.github.com>
Fix middleware reference conditions in Traefik IngressRoute
Document optional service and port properties for ingress paths
Copy link
Contributor

Copilot AI commented Dec 18, 2025

@akiyashkin-ronasit I've opened a new pull request, #32, to work on those changes. Once the pull request is ready, I'll request review from you.

@akiyashkin-ronasit
Copy link
Contributor Author

@eleonov I have resolved all comments from copilot.

@eleonov eleonov merged commit 1bfcf93 into main Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants