kiwigrid/helm-charts

[prometheus-thanos] Fix rendering when binary index headers are used

Closed this issue · 0 comments

Is this a request for help?:
No

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
Bug report

Version of Helm and Kubernetes:
Helm: v2.16.1
Kubernetes: v1.12.8

Which chart in which version:
prometheus-thanos v4.2.0

What happened:
Following on from #316, when opting in to binary index headers, the rendered Kubernetes objects are invalid. The current version of the helm template incorrectly chomps whitespace on the right, merging the flag --experimental.enable-index-header and the next line, ports: into one: --experimental.enable-index-headerports:

What you expected to happen:
The template should generate valid Kubernetes YAML

How to reproduce it (as minimally and precisely as possible):
Set storeGateway.binaryIndexHeader.enabled to true

Anything else we need to know:
I'm going to raise a PR to fix this.