Skip to content

Document optional service and port properties for ingress paths#30

Merged
akiyashkin-ronasit merged 2 commits intotraefik-ingressfrom
copilot/sub-pr-29
Dec 18, 2025
Merged

Document optional service and port properties for ingress paths#30
akiyashkin-ronasit merged 2 commits intotraefik-ingressfrom
copilot/sub-pr-29

Conversation

Copy link
Contributor

Copilot AI commented Dec 18, 2025

The ingress-route.yaml template supports optional service and port overrides per path, but these properties were not documented in values.yaml.

Changes

  • Added inline comments in values.yaml explaining optional path properties:
    • service: defaults to chart's fullname if omitted
    • port: defaults to service.port if omitted
  • Included example configuration demonstrating custom service/port usage

Example

ingress:
  hosts:
    - host: api.example.com
      paths:
        - path: /          # Uses default service and port
        - path: /api
          service: custom-api-service
          port: 8080

This clarifies the template's existing functionality without changing behavior.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…values.yaml

Co-authored-by: akiyashkin-ronasit <162979216+akiyashkin-ronasit@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 18, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • charts.bitnami.com
    • Triggering command: /usr/local/bin/helm helm repo add bitnami REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Update Traefik ingress implementation based on feedback Document optional service and port properties for ingress paths Dec 18, 2025
@akiyashkin-ronasit akiyashkin-ronasit marked this pull request as ready for review December 18, 2025 08:55
@akiyashkin-ronasit akiyashkin-ronasit merged commit 72f175c into traefik-ingress Dec 18, 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.

2 participants