Skip to content

Commit

Permalink
Update 0015-helm-value-style-variable-interfaces/README.md
Browse files Browse the repository at this point in the history
Co-authored-by: Austin Abro <[email protected]>
  • Loading branch information
Racer159 and AustinAbro321 authored Feb 4, 2025
1 parent 99cc648 commit 9dab5f9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion 0015-helm-value-style-variable-interfaces/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ zarf package deploy zarf-package-test.tar.zst --set MY_OVERRIDE_VARIABLE_NAME.ke

This will require some additional processing of the `zarf-config` files to allow them to be processed properly because of Viper's long-running [case-insensitive keys issue](https://github.com/spf13/viper/issues/1014). This could be performed similar to UDS CLI's implementation of this feature and / or by deprecating some of the existing less-used formats for configuration.

The `--set` syntax will change somewhat how variables are interpreted on the CLI (i.e. `--set VAR=100` will no monger represent `"100"` and instead will just be `100` internally). For `###` templates this can be mitigated by simply representing the value as a string for backwards compatibility though this will likely need to either be marked as breaking for Helm overrides or have additional flag changes / opt ins for this specific feature.
The `--set` syntax will change somewhat how variables are interpreted on the CLI (i.e. `--set VAR=100` will no longer represent `"100"` and instead will just be `100` internally). For `###` templates this can be mitigated by simply representing the value as a string for backwards compatibility though this will likely need to either be marked as breaking for Helm overrides or have additional flag changes / opt ins for this specific feature.

## Design Details

Expand Down

0 comments on commit 9dab5f9

Please sign in to comment.