Add missing appVersion to Chart.yaml for redis operator chart #107
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.
Problem
The current Helm chart for
operator-for-redisdoes not specify anappVersionfield inChart.yaml.Also
.Values.image.tagis empty invalues.yaml:As a result, the image tag in
deployment.yamldefaults to an empty value:This leads to an invalid image reference causing deployment failures.
According to the Helm documentation on the appVersion field, the
appVersionis used to track the application version being packaged, and it serves as the default image tag if not overridden.Solution
Add the
--app-version=$OPERATOR_VERSIONflag to the helm package command to fix this.This change ensures the chart's
appVersionis populated, preventing empty image tags and deployment issues.