Skip to content

Update helm template for deprecated APIs#12

Merged
jetstack-bot merged 1 commit into
cert-manager:masterfrom
rikatz:deprecated-apis
Jul 15, 2020
Merged

Update helm template for deprecated APIs#12
jetstack-bot merged 1 commit into
cert-manager:masterfrom
rikatz:deprecated-apis

Conversation

@rikatz
Copy link
Copy Markdown

@rikatz rikatz commented Jun 19, 2020

This changes the Helm Chart templates to remove deprecated APIs and use the stable ones.

@jetstack-bot jetstack-bot added dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 19, 2020
Signed-off-by: Ricardo Pchevuzinske Katz <ricardo.katz@serpro.gov.br>
@rikatz rikatz force-pushed the deprecated-apis branch from 81d61d5 to 4100cd1 Compare June 19, 2020 21:07
@jetstack-bot jetstack-bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. and removed dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. labels Jun 19, 2020
@rikatz
Copy link
Copy Markdown
Author

rikatz commented Jun 19, 2020

/assign @munnerz

@james-w
Copy link
Copy Markdown

james-w commented Jul 15, 2020

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
the versions that most people are going to be running in the wild.

Thanks,

James

@james-w
Copy link
Copy Markdown

james-w commented Jul 15, 2020

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!

@jetstack-bot jetstack-bot added the lgtm Indicates that a PR is ready to be merged. label Jul 15, 2020
@munnerz
Copy link
Copy Markdown
Member

munnerz commented Jul 15, 2020 via email

@munnerz munnerz added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 15, 2020
@jetstack-bot
Copy link
Copy Markdown
Contributor

[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

Details 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

@jetstack-bot jetstack-bot merged commit 46d50db into cert-manager:master Jul 15, 2020
fl0eb pushed a commit to fl0eb/cert-manager-webhook-strato that referenced this pull request Oct 18, 2025
Update helm template for deprecated APIs
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. 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/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants