service definition is missing admin port 8080

[dbacinski]

https://github.com/sftpgo/helm-chart/blob/main/sftpgo/values.yaml#L152

I had to patch service to make admin ui to work:

kubectl patch service sftpgo -n sftpgo --type='merge' -p '{
  "spec": {
    "ports": [
      {
        "port": 8080,
        "protocol": "TCP",
        "targetPort": 8080,
        "name": "admin"
      }
    ]
  }
}'

Also noticed that adding port to values.yaml and doing helm upgrade is not patching service

[drakkan]

Actually, you don't need to manually patch the service. The Admin UI (and the REST API) is already exposed by default through the http port.

By default, the chart maps the Service port 80 to the Container port 8080