Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 3, 2025

Users need to know that OCI Helm charts are published immediately upon release, while the legacy HTTP repository at https://charts.jetstack.io is delayed by a few hours.

Changes

Updated content/docs/installation/helm.md:

  • Added statement in the "Installing cert-manager with Helm" section highlighting OCI charts as the source of truth with immediate publication timing
  • Updated "Installing from the Legacy Helm Repository" section to note the multi-hour delay for HTTP repository updates

Screenshot

Updated Helm installation documentation showing OCI chart priority and timing information

The changes inform users to prefer OCI charts when timely access to new releases is required.

Original prompt

Create a PR to change one of the website's pages to tell users that the OCI helm charts are now the source of truth and that they are published immediately, and the old HTTP Helm charts will come a few hours after the OCI helm chart release.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

CyberArk tracker: VC-46548

@cert-manager-prow cert-manager-prow bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 3, 2025
@netlify
Copy link

netlify bot commented Nov 3, 2025

Deploy Preview for cert-manager ready!

Name Link
🔨 Latest commit ae856c3
🔍 Latest deploy log https://app.netlify.com/projects/cert-manager/deploys/690c7fd670cc5700084a4951
😎 Deploy Preview https://deploy-preview-1829--cert-manager.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI changed the title [WIP] Create a PR to change one of the website's pages to tell users that the OCI helm charts are now the source of truth and that they are published immediately, and the old HTTP Helm charts will come a few hours after the OCI helm chart release. Clarify OCI Helm charts as source of truth with immediate publication Nov 3, 2025
Copilot AI requested a review from maelvls November 3, 2025 11:11
Copilot finished work on behalf of maelvls November 3, 2025 11:11
@maelvls
Copy link
Member

maelvls commented Nov 3, 2025

@copilot Would you be able to draw a red box on what changed before/after in the screenshot? Or a side-to-side comparison?

Copy link
Contributor Author

Copilot AI commented Nov 3, 2025

@copilot Would you be able to draw a red box on what changed before/after in the screenshot? Or a side-to-side comparison?

Here's a side-by-side comparison with red boxes highlighting the changes:

Before/After Comparison

Two sections were updated:

  1. Section 1 - Added statement about OCI charts being source of truth with immediate publication
  2. Section 2 - Added warning about HTTP repository delay timing

Copilot finished work on behalf of maelvls November 3, 2025 11:47
@maelvls maelvls added the cybr Used by CyberArk-employed maintainers to report to line management what's being worked on. label Nov 3, 2025
@maelvls maelvls marked this pull request as ready for review November 3, 2025 15:44
@cert-manager-prow cert-manager-prow bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Nov 3, 2025
@maelvls maelvls force-pushed the copilot/fix-39950598-194756741-826214be-2604-4e1a-9e14-416d5bbd10b7 branch from e458917 to 601b69a Compare November 6, 2025 10:59
@maelvls
Copy link
Member

maelvls commented Nov 6, 2025

/approve
/lgtm

@cert-manager-prow cert-manager-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 6, 2025
@cert-manager-prow
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maelvls

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@maelvls maelvls force-pushed the copilot/fix-39950598-194756741-826214be-2604-4e1a-9e14-416d5bbd10b7 branch from 601b69a to ae856c3 Compare November 6, 2025 11:00
@cert-manager-prow cert-manager-prow bot removed lgtm Indicates that a PR is ready to be merged. dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. labels Nov 6, 2025
@cert-manager-prow
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@cert-manager-prow cert-manager-prow bot added the dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. label Nov 6, 2025
@maelvls maelvls added the lgtm Indicates that a PR is ready to be merged. label Nov 6, 2025
@cert-manager-prow cert-manager-prow bot merged commit 2f2b88f into master Nov 6, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cybr Used by CyberArk-employed maintainers to report to line management what's being worked on. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants