nginx annotations helm set syntax

[jansmets]

The only way I could set the ingress class was to run this:

--set ingress.annotations.'kubernetes\.io/ingress\.class'=bla \
warning: destination for annotations is a table. Ignoring non-table value <nil>

Perhaps it's better to do

--set ingress.annotations[0].key="kubernets.io/ingress.class" \
--set ingress.annotations[0].value="bla" \

(helm 1.9)

[jainishshah17]

@jansmets Yah that's the right way to pass it using --set flag.

[jansmets]

I hope you agree that this is quite annoying and the warning makes it even dubious.

[danielezer]

@jansmets I think this syntax is the standard for adding annotations, just passing them as is into the spec, since the annotations object is a map and not a dictionary. I think you should use a values file if this syntax bothers you

[bechampion]

This worked for me , i can't seem to find anywhere on the interwebs:
helm install stable/nginx-ingress --set controller.service.annotations."cloud\.google\.com\/load-balancer\-type"=Internal

[evannorstrand]

@danielezer - the fix to this is to add a {} after every annotation location in values.yaml. Should look like below:

annotations: {}

[danielezer]

Thanks @evannorstrand! I see you added this fix to your PR. Thanks for your contribution!!