tilt icon indicating copy to clipboard operation
tilt copied to clipboard

Support helm `set-string` parameter in `helm`

Open jazzdan opened this issue 4 years ago • 2 comments

Due to the magic of YAML and dynamic typing it can be difficult to force a field to be of a certain type. For example if you use --set to set foo to "true" "true" will actually become the integer 1. This problem is described more in depth here.

To get around this Helm has a --set-string flag that forces the value that is being set to be a string. We should add support for this as a parameter to the helm builtin.

jazzdan avatar Jul 07 '20 14:07 jazzdan

You can do this with helm_resource's flags param: https://github.com/tilt-dev/tilt-extensions/tree/master/helm_resource

I'm fine leaving this issue open to add set_string to the old helm() built-in, but I think most people would be happier with helm_resource

nicks avatar Jan 18 '22 19:01 nicks

Should helm_remote be updated to use helm_resource instead of the built-in helm then? Since it too would have the same issue

electrofelix avatar Nov 22 '22 15:11 electrofelix