Change how we update the Helm Chart repo index.yaml
#8479
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.
Type of change
Description
This PR addresses #8437 by changing how we manage and update the Helm Chart repository
index.yaml
. Today, theindex.yaml
is part of this repository. So when we do the release, we have to update theindex.yaml
with the new release before we tag the release and run the CI. But since the Helm Chart archive is built during the CI run from the Git sources, it is generated only after theindex.yaml
is updated. So as a result, the checksum does not match.This PR removes the
index.yaml
from the operators repo and we will keep it only on the website where the repo is hosted. Theindex.yaml
will be updated manually based on the Helm Chart archive from the CI build and thus will have the correct checksum. the necessary steps are described in the release docs.This resolves #8437 - however, it does not fix the old signatures. It just makes sure the next one will be correct.
This PR also fixes two other minor issues:
Checklist