-
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
🌱 Add valid helm chart structure and helm templating support for the chart #60
🌱 Add valid helm chart structure and helm templating support for the chart #60
Conversation
0e47aad
to
ab495b3
Compare
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
ab495b3
to
96d1a07
Compare
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.
overall looks good, does it work with make dev-env
?
Yes, it does, the charts/default directory is only used for that |
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.
lgtm
What this PR does / why we need it:
Based on #55, depends on #58
Cleaning up and restructuring release process to provide a valid helm charts structure and templating. This allows a valid tag substitution in each release. Example release generated by this: https://github.com/Danil-Grigorev/rancher-turtles/releases/tag/rancher-turtles-1.2.29
To try it out:
helm repo add rancher-turtles-testing https://danil-grigorev.github.io/rancher-turtles helm repo update helm install test rancher-turtles-me/rancher-turtles --version 1.2.29
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Checklist: