-
Notifications
You must be signed in to change notification settings - Fork 703
site/guides: Add cert-manager example to explain how to use Certificate resources #3175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
site/guides: Add cert-manager example to explain how to use Certificate resources #3175
Conversation
|
Blocked behind #3172 |
Codecov Report
@@ Coverage Diff @@
## main #3175 +/- ##
=======================================
Coverage 74.00% 74.00%
=======================================
Files 96 96
Lines 6001 6001
=======================================
Hits 4441 4441
Misses 1464 1464
Partials 96 96 |
11f99cf to
adbc379
Compare
skriss
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple minor language comments, but looks pretty good
…te resources Update the cert-manager guide to explain how to use cert-manager Certificate resources instead of Ingress shim resources to request certificates via cert-manager. Fixes projectcontour#2709 Signed-off-by: Steve Sloka <[email protected]>
Signed-off-by: Steve Sloka <[email protected]>
Signed-off-by: Steve Sloka <[email protected]>
Signed-off-by: Steve Sloka <[email protected]>
83739ff to
795391c
Compare
skriss
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
Update the cert-manager guide to explain how to use cert-manager Certificate resources
instead of Ingress shim resources to request certificates via cert-manager.
Fixes #2709
Fixes #2018
Signed-off-by: Steve Sloka [email protected]