helm: update chart repo to https://kubernetes-sigs.github.io/secrets-store-csi-driver/charts

[aramase]

Signed-off-by: Anish Ramasekar [email protected]

What this PR does / why we need it:

• The helm charts have been moved to https://kuberentes-sigs.github.io/secrets-store-csi-driver/charts. With this PR, we're updating the install and upgrade instructions to use the new helm chart repository URL.

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #

Special notes for your reviewer:

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aramase

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:

• OWNERS [aramase]

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

[aramase]

The preview is available here:

• https://deploy-preview-695--kubernetes-sigs-secrets-store-csi-driver.netlify.app/getting-started/installation.html
• https://deploy-preview-695--kubernetes-sigs-secrets-store-csi-driver.netlify.app/getting-started/upgrades.html

/hold
for reviews

[nilekhc]

/lgtm

[tam7t]

nit: I think it may be fine to remove this note from all the first time install pages and only leave it on the upgrade one, it may 'get in the way' of new users who have to read it and determine whether it matters to them or not.

[aramase]

I was thinking the same, but only concern is what if they have the repo added before and plan to install it now. They might reference the doc for the configuration parameters and see the note to change the URL? wdyt?

[tam7t]

Ah that makes sense, maybe we have a bug for like 3-6 months to just remove this note just so we dont keep it around forever

[aramase]

sgtm! Issue: #699

[aramase]

/retest

[aramase]

/test pull-secrets-store-csi-driver-e2e-vault

[tam7t]

/lgtm

[ritazh]

Let's make sure the release notes include a big warning that the helm chart repo has moved.

[aramase]

Let's make sure the release notes include a big warning that the helm chart repo has moved.

sounds good! Thanks, Rita!

[aramase]

/hold cancel

[aramase]

/test pull-secrets-store-csi-driver-e2e-windows

[fableforester]

@aramase
The new URL (https://kuberentes-sigs.github.io/secrets-store-csi-driver/charts) is not available and leads into a 404 Not Found. We are currently unable to get the driver running.

[aramase]

@fableforester I just tried and the url works fine:

➜ helm repo rm secrets-store-csi-driver
"secrets-store-csi-driver" has been removed from your repositories

~
➜ helm repo add secrets-store-csi-driver https://kubernetes-sigs.github.io/secrets-store-csi-driver/charts
"secrets-store-csi-driver" has been added to your repositories

[fableforester]

@aramase
Weird, must have been a temporary unavailability yesterday. The command is working now, sorry!

[mikestef9]

Typo in docs "kuberentes" instead of "kubernetes"

[nilekhc]

Typo in docs "kuberentes" instead of "kubernetes"

@mikestef9 Good catch! Thanks. Created a PR to fix this.

@tam7t Could you please update release notes?

[aramase]

@mikestef9 Thanks for reporting the issue.

@tam7t Could you please update release notes?

The typo has been fixed in the release notes.