Error: YAML parse error on hoppscotch/templates/ingress.yaml: error converting YAML to JSON

[boindil]

When using service.tls.enabled = true an error occurs because {{- toYaml .Values.service.ingress.annotations | nindent 4 }} returns null in the generated yaml.

[jonathanfoster]

This issue is resolved with PR #35.

[jonathanfoster]

@boindil This should be fixed in hoppscotch-0.1.0. Mind giving it a try to confirm?

[12345ieee]

No, it's not fixed, you need to merge #24 (or equivalent change).

[jonathanfoster]

@12345ieee It's fixed in the hoppscotch chart, not the shc or she charts. The hoppscotch chart combines the two so it's easier to maintain with just one place to make a change.

[12345ieee]

Ah, I followed the readme and installed one of the old charts.

Consider changing it, monday I'll test the combined chart.

[jonathanfoster]

That's a good idea. Maybe we give this new chart a bit of time to work out any kinks then make the switch. Let me know how it goes for you.

[12345ieee]

I've just deployed it, definitely a step up in functionality and documentation.

My only annoyance is that the AIO pod logs all the readiness probes pings with:
Backend Server | GET /ping 200 - 0.686 ms
which floods the logs.

The only other change I added was

aio:
  readinessProbe:
    initialDelaySeconds: 10

because the default of 0 would kill the pod sometimes.

Nice work.

[jonathanfoster]

Good call on setting the initial delay. I get at least one restart on each deploy with it set to 0.