Skip to content

fix(helm): pass arrays via per-element --set-string args#20

Merged
danilovid merged 1 commit into
Runtime-Radar:mainfrom
danilovid:fix/helm-set-arrays-per-element
May 15, 2026
Merged

fix(helm): pass arrays via per-element --set-string args#20
danilovid merged 1 commit into
Runtime-Radar:mainfrom
danilovid:fix/helm-set-arrays-per-element

Conversation

@danilovid
Copy link
Copy Markdown
Collaborator

Helm's --set treats ["a","b"] as a literal string, so generating
--set 'admission-controller.ignoredNamespaces=["a","b"]' failed chart
schema validation with Expected: array, given: string.

Refactor buildHelmArgs to recurse into slice/array elements and emit
one --set-string 'name[i]=value' per element. The same path also
correctly handles arrays of structs (e.g. notifier.overwriteEnv),
which the previous JSON-marshaled form already mis-encoded.

@danilovid danilovid self-assigned this May 14, 2026
@danilovid danilovid requested review from avlllo and torkel-hoge May 15, 2026 10:28
@danilovid danilovid merged commit 321ad7d into Runtime-Radar:main May 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant