42564
39%

Ran 03 Jun 2022 09:09AM UTC

Jobs 2

Files 941

Run time 7min

Badge

Embed ▾

Committed 03 Jun 2022 08:34AM UTC coverage: 33.754% (+0.01%) from 33.741%

Build # 42564

Build Type

push

travis-ci-com

Committed by

joamaki

Commit Message

helm: Fix syntax error in Hubble UI className

The ingressClassName in the Hubble UI chart accidentally used the `{{-`
template syntax which trims any leading whitespace, resulting in the
line being rendered as `ingressClassName:foo`, which is not valid YAML,
as YAML mandates a whitespace character between the key and value.

This fixes the issue where when the className was specified, Helm would
fail with the following error:

```
Error: Failed to render chart: exit status 1: Error: YAML parse error on cilium/templates/hubble-ui/ingress.yaml: error converting YAML to JSON: yaml: line 10: mapping values are not allowed in this context
```

Fixes: #20054
Fixes: 17f42b3b19e4 ("feat(helm): add ingressClassName to ingress spec")

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>

Coverage Stats

47704 of 141330 relevant lines covered (33.75%)

3752.3 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	42564.1	03 Jun 2022 09:15AM UTC	0	33.75	Travis Job 42564.1
2	42564.2	03 Jun 2022 09:09AM UTC	0	33.75	Travis Job 42564.2

Source Files on build 42564

Detailed source file information is not available for this build.

cilium / cilium / 42564
39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 42564

cilium / cilium / 42564 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 42564

cilium / cilium / 42564
39%

README BADGES
x