Skip to content
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

Change how we update the Helm Chart repo index.yaml #8479

Conversation

scholzj
Copy link
Member

@scholzj scholzj commented May 4, 2023

Type of change

  • Bugfix

Description

This PR addresses #8437 by changing how we manage and update the Helm Chart repository index.yaml. Today, the index.yaml is part of this repository. So when we do the release, we have to update the index.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 the index.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. The index.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

  • Update documentation
  • Reference relevant issue(s) and close them after merging

@scholzj scholzj added this to the 0.35.0 milestone May 4, 2023
@scholzj scholzj requested a review from Frawless May 4, 2023 21:23
Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj scholzj force-pushed the make-sure-helm-chart-repo-will-have-the-right-digest branch from b30f0bf to 8de9a12 Compare May 4, 2023 21:36
@scholzj scholzj merged commit e1cf529 into strimzi:main May 5, 2023
@scholzj scholzj deleted the make-sure-helm-chart-repo-will-have-the-right-digest branch May 5, 2023 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Wrong digest of the Cluster Operator Helm Chart in the index.yaml
2 participants