Helm chart: Add garage.existingConfigmap and replace garage.garage.toml with garage.garageTomlString #923

Merged

maximilien merged 2 commits from jessebot/garage:allow-existing-configmap into main

2025-01-15 23:53:25 +00:00

jessebot commented

2025-01-09 11:21:01 +00:00

Contributor

This fixes #922

Helm chart updates

New helm parameter, garage.existingConfigmap to use an existing ConfigMap for garage.toml:
```
garage:
  existingConfigmap: ""
```
Added some autogenerated docs using helm-docs. It works by processing every comment that starts with # -- I'm a comment as a description for the parameter it is directly above. This gives a small chart in a README that users can reference in the same directory as the chart.

Changes

Replaces garage.garage.toml with garage.garageTomlString. You can still use the old feature like this:
```
garage:
  garageTomlString: |-
    myconfig = "somegreatconfig"    
```
This also moves all the configuration back to the ConfigMap template file so that no GoTemplating happens in the values.yaml itself.

This fixes https://git.deuxfleurs.fr/Deuxfleurs/garage/issues/922 **Helm chart updates** - New helm parameter, `garage.existingConfigmap` to use an existing ConfigMap for garage.toml: ```yaml garage: existingConfigmap: "" ``` - Added some autogenerated docs using helm-docs. It works by processing every comment that starts with `# -- I'm a comment` as a description for the parameter it is directly above. This gives a small chart in a README that users can reference in the same directory as the chart. **Changes** - Replaces `garage.garage.toml` with `garage.garageTomlString`. You can still use the old feature like this: ```yaml garage: garageTomlString: |- myconfig = "somegreatconfig" ``` This also moves all the configuration back to the ConfigMap template file so that no GoTemplating happens in the values.yaml itself.

jessebot added 1 commit 2025-01-09 11:21:02 +00:00

Add garage.existingConfigmap and replace garage.garage.toml with garage.garageTomlString

ci/woodpecker/pr/debug Pipeline was successful

Details

b94acb1114

also moves all gotemplating back to configmap

and adds autogenerated docs via helm-docs

Signed-off-by: jessebot <jessebot@linux.com>

jessebot referenced this pull request

2025-01-09 11:24:54 +00:00

gotemplating in helm chart causes errors with Argo CD deployments #922

jessebot changed title from ~~Add garage.existingConfigmap and replace garage.garage.toml with garage.garageTomlString~~ to Helm chart: Add garage.existingConfigmap and replace garage.garage.toml with garage.garageTomlString

2025-01-09 11:34:41 +00:00