Migrate from legacy Helm to OCI #516
Conversation
cert-manager-prow
bot
added
the
dco-signoff: yes
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
cert-manager-prow
bot
added
the
size/M
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the cert-manager Helm chart references from the legacy HTTPS repository (https://charts.jetstack.io) to the OCI registry (oci://quay.io/jetstack/charts/cert-manager), aligning with modern Helm practices and the changes made in the approver-policy repository.
Key changes:
- Updated Helm installation command to use OCI registry format with proper argument ordering
- Removed manual pre-release checks for upstream CSI images (now automated via Renovate)
- Clarified that the legacy HTTP repository is maintained by Venafi and may experience delays
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| make/test-e2e.mk | Migrated cert-manager Helm installation from legacy HTTP repo to OCI registry, updated version variable reference, and corrected command syntax for OCI charts |
| RELEASE.md | Updated release documentation to reflect OCI-first approach, removed obsolete manual image update checks, clarified legacy repo status, and fixed minor grammar/spelling issues |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Erik Godding Boye <[email protected]>
1 participant
This PR mimics the changes done in cert-manager/approver-policy#748 to update our release process documentation. In addition, I have also migrated other references to the legacy https Helm chart repo to OCI. And also removed the pre-release step to check for new versions of upstream CSI images as this is done "continously" by Renovate now.