Implement packaging for helm deployment #682
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A note for reviewers: this is a proposal. If you give the green light to it, I will add required tests and documentation.
This commit adds a possibility to package a helm chart before deployment
using
helm package
command. This can be useful to set a specific versionfor a chart.
One of the workflows that skaffold assumes is to deploy a Helm chart that
resides within some directory in the project folder.
Chart.yaml
file inthat folder is basically static file, and
--version
flag inhelm install ./path/to/chart --version
command actually does nothing,so does "version:" field in helm section of skaffold.yaml.
To set a chart version dynamically, we have to use
helm package
commandsetting
--version
(and optionally--app-version
) flags accordingly.This commit adds a new section
packaged:
to helm section skaffold.yamlthat can be used to configure dynamic chart versions.