Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configuration file for server doesn't allow unexpected values #18

Open
TomHellier opened this issue Sep 16, 2022 · 1 comment
Open

Configuration file for server doesn't allow unexpected values #18

TomHellier opened this issue Sep 16, 2022 · 1 comment

Comments

@TomHellier
Copy link
Contributor

I'm attempting to use this chart with the latest server, and it is expecting different values in the configuration file for the server.

It is currently impossible to use with the current chart as it is difficult to clear maps: helm/helm#5184

I will add ability to fully override the contents of the configuration file

TomHellier added a commit to TomHellier/helm-charts-parca-dev that referenced this issue Sep 16, 2022
the server validates the configuration file, and if it sees values it does not accept, the process exits.

This change will allow users to force overwrite the values which exist in the helm chart to stop the helm maps merging
TomHellier added a commit to TomHellier/helm-charts-parca-dev that referenced this issue Sep 16, 2022
@morremeyer
Copy link
Contributor

@TomHellier You can override with ~ instead of null in current versions of helm to remove a map key.

The server config should allow for arbitrary values, you can add any key you want to .Values.server.config.

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

No branches or pull requests

2 participants