nginx annotations helm set syntax about charts HOT 6 CLOSED

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

from charts.

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

from charts.

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

from charts.

@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

from charts.

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

annotations: {}

from charts.

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

from charts.