Skip to content

Add serviceAccountName to issuers CRDs for http01 solvers#3139

Merged
jetstack-bot merged 1 commit intocert-manager:masterfrom
paulwilljones:feature/issuer-serviceaccount
Aug 3, 2020
Merged

Add serviceAccountName to issuers CRDs for http01 solvers#3139
jetstack-bot merged 1 commit intocert-manager:masterfrom
paulwilljones:feature/issuer-serviceaccount

Conversation

@paulwilljones
Copy link
Copy Markdown

@paulwilljones paulwilljones commented Aug 3, 2020

What this PR does / why we need it:
This adds support for the serviceAccountName field for HTTP01 solvers.

Which issue this PR fixes:
fixes: #2770

Release note:

Add `serviceAccountName` field to `podTemplate` for ACME HTTP01 issuers

/kind feature

Signed-off-by: Paul Jones <paul.jones@jetstack.io>
@jetstack-bot jetstack-bot added kind/feature Categorizes issue or PR as related to a new feature. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Aug 3, 2020
@jetstack-bot jetstack-bot requested a review from meyskens August 3, 2020 14:37
@jetstack-bot jetstack-bot added area/acme Indicates a PR directly modifies the ACME Issuer code area/acme/http01 Indicates a PR modifies ACME HTTP01 provider code size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/api Indicates a PR directly modifies the 'pkg/apis' directory area/deploy Indicates a PR modifies deployment configuration release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Aug 3, 2020
@meyskens
Copy link
Copy Markdown
Contributor

meyskens commented Aug 3, 2020

A familiar face!

Looks great! Thanks for this!

/approve
/lgtm

@jetstack-bot jetstack-bot added the lgtm Indicates that a PR is ready to be merged. label Aug 3, 2020
@jetstack-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: meyskens, paulwilljones

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 added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 3, 2020
@jetstack-bot jetstack-bot merged commit 6e17f55 into cert-manager:master Aug 3, 2020
@jetstack-bot jetstack-bot added this to the v1.0 milestone Aug 3, 2020
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. area/acme/http01 Indicates a PR modifies ACME HTTP01 provider code area/acme Indicates a PR directly modifies the ACME Issuer code area/api Indicates a PR directly modifies the 'pkg/apis' directory area/deploy Indicates a PR modifies deployment configuration dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow HTTP01 solver pod to use non-default ServiceAccount

3 participants