stable/cert-manager: deprecate cert-manager chart

[munnerz]

What this PR does / why we need it:

Deprecates the cert-manager chart in favour of the new chart hosted by Jetstack on Helm Hub.

Checklist

• DCO signed
• Chart Version bumped
• Variables are documented in the README.md

[munnerz]

@unguiculus this feels like a weird requirement, but the e2e checker fails if I leave maintainers here 😬

[munnerz]

According to the Helm chart docs, after this version of the chart is published to the stable repo, we should also remove this chart from the repository altogether. Is that still correct? https://helm.sh/docs/developing_charts/#deprecating-charts

---

When managing charts in a Chart Repository, it is sometimes necessary to deprecate a chart. The optional deprecated field in Chart.yaml can be used to mark a chart as deprecated. If the latest version of a chart in the repository is marked as deprecated, then the chart as a whole is considered to be deprecated. The chart name can later be reused by publishing a newer version that is not marked as deprecated. The workflow for deprecating charts, as followed by the helm/charts project is:

• Update chart’s Chart.yaml to mark the chart as deprecated, bumping the version
• Release the new chart version in the Chart Repository
• Remove the chart from the source repository (e.g. git)

[munnerz]

Tests here are failing as the chart e2e test cluster doesn't have the cert-manager CRDs installed (this is a recurring problem 🙈)

@prydonius any chance you can re-create these resources so we can get this (final!) PR merged? 😄

[munnerz]

(... plus a /lgtm and a /retest shortly afterwards, so that this actually makes it in before the cluster gets recycled again 😅)

[unguiculus]

/lgtm

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: munnerz, unguiculus

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• stable/cert-manager/OWNERS [munnerz,unguiculus]

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

[unguiculus]

/retest

[saplla]

@munnerz Please can you update the README so the deprecation is obvious. Thanks

[drewwells]

Tests here are failing as the chart e2e test cluster doesn't have the cert-manager CRDs installed (this is a recurring problem 🙈)

@prydonius any chance you can re-create these resources so we can get this (final!) PR merged? 😄

Tests should not require hand curating resources on the environment. This is the purpose of handling the lifecycle of chart resources in the chart instead of ad-hoc processes.