bjw-s / helm-charts

A collection of Helm charts
https://bjw-s.github.io/helm-charts/
Apache License 2.0
535 stars 100 forks source link

When using Route.main.hostnames with templated values it returns an error #211

Closed snoopy82481 closed 8 months ago

snoopy82481 commented 9 months ago

Details

What steps did you take and what happened:

When using the following configuration the value for Release.Name is not replaced in the hostnames section

route:
  main:
    enabled: true
    parentRefs:
      - name: cilium-internal
        namespace: networking
        sectionName: *app
    hostnames:
      - "{{ .Release.Name }}.${SECRET_DOMAIN}"
    rules:
      backendRefs:
        - port: http

What did you expect to happen:

I expected the Release.Name to be replaced when installing/upgrading deployment.

Anything else you would like to add:

Variable substitutions still works so the ${SECRET_DOMAIN} is replaced.

Additional Information:

bjw-s commented 8 months ago

Turns out templates weren't supported yet for this field. I've added the functionality to v2.1.0