-
Notifications
You must be signed in to change notification settings - Fork 18
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
Helm chart bring your own config.toml #23
Comments
I came here to ask for the same feature. The problem with using environment variables in values.yaml is that you can't use templates with them. This is forcing me to use fixed resource names and YAML anchors. Being able to generate a custom config file with a template and have it mounted into the container would solve this problem. |
I gave it a shot, but would love to get your feedback before publishing. Can you give it a try? I figured that if you want to provide a full |
@mxk or @drewwells : does my approach suit your needs? Have you had a chance to give it a try? |
Thanks for looking into this! I think that would work, but in my case, I found it easier to define my own Athens deployment rather than using the chart. It simplified the config, allowed me to provide other files, and generally gave me more control over how Athens is deployed. In my use case, Athens is an internal implementation detail, so I'm probably not representative of most of the users. |
Is your feature request related to a problem? Please describe.
The config.yaml is hardcoded into the docker container. This is fine, but the helm chart needs a way to override this config file.
Describe the solution you'd like
Option to mount an externally provided config file. I think most options are available as environment variable overrides, but the athens process does not log its config. It's unclear what the current settings are from the combination of config file and env variable overrides.
Describe alternatives you've considered
A long list of helm values to override all settings inside the container. nginx does this https://github.com/helm/charts/tree/master/stable/nginx-ingress#configuration
Additional context
The text was updated successfully, but these errors were encountered: