Error installing the chart

[bab5470]

Getting the following when trying to install the chart:

ERROR parsing the configuration file: '/etc/krakend/krakend.json': 1 error(s) decoding:

* cannot parse 'port' as int: strconv.ParseInt: parsing "tcp://10.43.126.100:80": invalid syntax

I found this which seems to be similar: mattermost-community/focalboard#363

The solution proposed is to add: enableServiceLinks: false to the pod template spec to get around this. Will give this a try but FYI to anyone else that encounters this and to the helm chart author.

[bab5470]

Actually maybe that's not such a good idea. Just found this:

According to the source code:

We always want to add environment variabled for master services from the master service namespace, even if enableServiceLinks is false.

Which basically means that you will and should not prevent those specific environment variables from being injected to the pods.

[bab5470]

Any other ideas or suggestions on how we solve the above issue?

[reytech-dev]

Hey there,
I did the following:

I added a new attribute here: https://github.com/mikescandy/krakend-helm/blob/master/values.yaml#L41

Something like that:

...
service:
  type: ClusterIP
  port: 80
  name: krakend-svc
...

I updated https://github.com/mikescandy/krakend-helm/blob/master/templates/service.yaml#L4

...
apiVersion: v1
kind: Service
metadata:
  name: {{ .Values.service.name }}
  labels:
    {{- include "krakend.labels" . | nindent 4 }}
...

And I updated https://github.com/mikescandy/krakend-helm/blob/master/templates/ingress.yaml#L2

{{- if .Values.ingress.enabled -}}
{{- $fullName := Values.service.name . -}}
{{- $svcPort := .Values.service.port -}}
{{- if semverCompare ">=1.14-0" .Capabilities.KubeVersion.GitVersion -}}
apiVersion: networking.k8s.io/v1beta1
...

Like that there is no name collision anymore.
Since krakend ist usually directly connected to the ingress resource, you don't have to change anything except the mentioned configurations.
Not a perfect solution but keeps the changes very small and works.

[ksylvan]

@bab5470 there is a newer maintained repo here for the Helm chart: https://github.com/equinixmetal-helm/krakend
Maybe you can try that?