Skip to content

Commit

Permalink
Merge pull request #1432 from ThatsMrTalbot/docs/update-releasing-doc…
Browse files Browse the repository at this point in the history
…s-with-chart-test

docs: add step to test helm chart before publishing
  • Loading branch information
jetstack-bot authored Feb 28, 2024
2 parents 5429089 + 79580ab commit b6506e6
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions content/docs/contributing/release-process.md
Original file line number Diff line number Diff line change
Expand Up @@ -602,8 +602,14 @@ page if a step is missing or if it is outdated.
1. [Visit the pull request](https://github.com/jetstack/jetstack-charts/pulls)
2. Review the changes
3. Fix any failing checks
4. Merge the PR
5. Check that the [cert-manager Helm chart is visible on ArtifactHUB](https://artifacthub.io/packages/helm/cert-manager/cert-manager).
4. Test the chart
1. Download the chart tarball from the pull-request
2. Start a new local Kind cluster `kind create cluster --name release`
3. Install the helm chart onto the kind cluster `helm install cert-manager ./cert-manager-v0.14.2.tgz --set installCRDs=true -n cert-manager`
4. Ensure install succeeds and all components are running
5. Tear down the kind cluster `kind delete cluster --name release`
5. Merge the PR
6. Check that the [cert-manager Helm chart is visible on ArtifactHUB](https://artifacthub.io/packages/helm/cert-manager/cert-manager).
15. **(final + patch releases)** Merge the 4 Website PRs:
Expand Down

0 comments on commit b6506e6

Please sign in to comment.