Allow service setting externalName about helm-charts HOT 5 CLOSED

I'm pushing out 3.3.0 right now and the required app-template (schema) updates after that, so you should be good to use that in a few minutes :)

from helm-charts.

Thanks for the effort, is there a way to test this implementation in my helm chart? It does not looks like you build beta/unstable charts?

I also tried to add the template-service to my templates dir, but I was not able to fetch all variables from common template, which needs to be used in annotations and labels. Maybe you have an enable for that?

from helm-charts.

You didn't write a test for this externalName setting. Are you sure it was neigh to add it to the json file?
Because the template generation does not include this.

service:
    main:
      primary: true
      ... snip ...
    api-backend:
      primary: false
      controller: app
      type: ExternalName
      externalName: httpbin.org
      ports: 
        http:
          port: 80

returns the following spec

spec:
  type: ExternalName
  ports:
    - port: 80
      targetPort: 80
      protocol: TCP
      name: http

so without the externalName. The schema check does not complain about the field.

from helm-charts.

Wh00ps... I interpreted the bug report as if it was only the schema preventing it from working. Will push a fix later today Just pushed 3.3.1 which should fix the issue

from helm-charts.

Thanks, it works now! 🥇

from helm-charts.