Update helm template for deprecated APIs#12
Conversation
Signed-off-by: Ricardo Pchevuzinske Katz <ricardo.katz@serpro.gov.br>
|
/assign @munnerz |
|
Hi, thanks for the contribution. When were those stable APIs added? This is just an example so we don't have to worry about compatibility too much, but it's good to be compatible with Thanks, James |
|
Looking at 1.14: It has
The others are the cert-manager.io APIs, where we can reasonably assume that people are up to date, so /lgtm Thanks! |
|
/approve
…On Wed, 15 Jul 2020 at 09:03, James Westby ***@***.***> wrote:
Looking at 1.14:
It has
- apiregistration.k8s.io/v1:
https://v1-14.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.14/#apiservice-v1-apiregistration-k8s-io
- apps/v1:
https://v1-14.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.14/#apps-v1
- rbac.authorization.k8s.io/v1:
https://v1-14.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.14/#rolebinding-v1-rbac-authorization-k8s-io
The others are the cert-manager.io APIs, where we can reasonably assume
that people are up to date, so
/lgtm
Thanks!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABRWP4ZETLQS5B2LJTSRHLR3VPDRANCNFSM4ODBGR3Q>
.
|
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: munnerz, rikatz The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Update helm template for deprecated APIs
This changes the Helm Chart templates to remove deprecated APIs and use the stable ones.