concourse/concourse-chart

web.service.api.NodePort not honored

DonBower opened this issue · 1 comments

In the charts/templates/web-api-svc.yaml file, line 39 checks for the existence of a value at .Values.web.service.api.atcNodePort, and then sets the nodePort: value on line 40 to .Values.web.service.api.NodePort.
It should check for the existence of .Values.web.service.api.NodePort, not .atcNodePort

Documentation for NodePort says:
Sets the nodePort for api when using NodePort

It Does not mention atcNodePort.

This workaround produces a NodePort service at 30001, which is what is desired for my case:

web:
  replicas: 3
  service:
    api:
      type: NodePort
      atcNodePort: 30008
      NodePort: 30001