-
Notifications
You must be signed in to change notification settings - Fork 9
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
Publish helm chart using GH pages #16
Conversation
Signed-off-by: David Young <davidy@funkypenguin.co.nz>
Signed-off-by: David Young <davidy@funkypenguin.co.nz>
Signed-off-by: David Young <davidy@funkypenguin.co.nz>
Signed-off-by: David Young <davidy@funkypenguin.co.nz>
Add support for initContainer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool. Good idea
|
||
## InitContainers allows injecting additional initContainers. This is meant to allow doing some changes | ||
## (permissions, dir tree) on mounted volumes before starting prometheus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
starting prometheus?
icon: https://avatars.githubusercontent.com/u/88781313?s=200&v=4 | ||
maintainers: | ||
- name: funkypenguin | ||
email: davidy@funkypenguin.co.nz | ||
annotations: | ||
artifacthub.io/changes: |- | ||
- kind: added | ||
description: Initial release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you copy that over from another repo? The home and icon are wrong. I think having your name as maintainer is misleading as well since you do not have repo access
volumes: | ||
{{- toYaml .Values.volumes | nindent 8 }} | ||
initContainers: | ||
{{ toYaml .Values.extraInitContainers | indent 8 }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indent seems wrong. Would look nicer with nindent like the other attributes. Also the value name is not extraInitContainers
{{ toYaml .Values.extraInitContainers | indent 8 }} | |
{{- toYaml .Values.initContainers | nindent 8 }} |
Signed-off-by: David Young <davidy@funkypenguin.co.nz>
Oh and I got mixed up with the two PRs being open. The initContainers feature shouldn't be here, I think it got added from the other PR/branch by mistake. If you were thinking of bundling the two PRs, I'd rather have two well defined PRs |
Signed-off-by: David Young <davidy@funkypenguin.co.nz>
Fix bad variable name
Oops, you're right, the initContainers shouldn't be here. I'll close this PR and open a clean one! |
Hey guys!
Here's a PR to add some GitHub Actions to auto-publish the helm chart, using GitHub Pages. This means that people who use the chart in CI/CD, for example, and who can't do an install directly from helm, can still install a specific version from a URL (in this case, the chart repo would be https://grafana.github.io/flagger-k6-webhook/)
You can test it against the version I published in my fork, at https://funkypenguin.github.io/flagger-k6-webhook/ ;)
D