This repository has been archived by the owner on Mar 24, 2023. It is now read-only.
make helmValues work in non init-chart workflows #858
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.
What I Did
I made HelmValues work in non init-chart workflows.
full POC (also in integration test):
How I Did it
daemontypes.Step.HelmValues
, copy it over from existingapi.Step.HelmValues
HelmValues.Path
field inGET /navcycle/step/helm-values
and thePOST /helm-values
, falling back to the old way of.ship/tmp/values.yaml
filepath.Dir(step.HelmValues.path)
as the base directory for linting, defaulting to the old.ship/tmp/chart
How to verify it
ship init
the ship.yml in integration/base/shipapp-helm-values/input/.ship/ship.yamlDescription for the Changelog
Make some changes around path handling that allow a
lifecycle.v1.helmValues
step to be used outside of the coreship init <chart>
workflow.