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

Helm chart QoL improvements #884

Closed
vordimous opened this issue Mar 27, 2024 · 1 comment · Fixed by #901
Closed

Helm chart QoL improvements #884

vordimous opened this issue Mar 27, 2024 · 1 comment · Fixed by #901
Assignees
Labels
story Feature description from user's perspective

Comments

@vordimous
Copy link
Contributor

vordimous commented Mar 27, 2024

Describe the desired outcome from the user's perspective
Helm charts can have a wide range of usage patterns that are exposed through the values.yaml file. There are some common components and helm templates to add that can help deploy zilla in situations devs are more familiar with.

ENV vars are the main wait to dynamically modify the zilla.yaml and giving the user more options to configure them will be much more helpful. If we can just replicate what the Grafana helm chart has it would make any deployment pattern easier.

Acceptance criteria

  • use the structure of chart templates and values found in the default helm create zilla template
    • unique name use values generated in the _helpers.tpl file
    • keep optional ingress and service account
    • keep values.yaml structure and similar as possible with comments
  • remove license text from the NOTES.txt file
  • add comments in values.yaml for added parameters
  • add more ways to configure ENV vars
@vordimous vordimous added the story Feature description from user's perspective label Mar 27, 2024
@vordimous
Copy link
Contributor Author

moved HTTP/2 feature to its on ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
story Feature description from user's perspective
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants