garden
garden copied to clipboard
Deprecate cert-manager integration
Bug
Current Behavior
The current cert-manager
version supported in the Garden integration is way too old (v0.11.0
) and needs an update.
Expected behavior
As a user I want to be able to generate certificates using the latest version of cert-manager
Reproducible example
N/A
Workaround
You could potentially install cert-manager
manually and use it to manage tlsCertificates but we haven't tested with the latest cert-manager
version.
Suggested solution(s)
Additional context
Your environment
- OS:
- How I'm running Kubernetes:
garden version
We've decided to deprecate the cert-manager integration and now recommend that users set up cert-manager separately. What we'd need to do is make sure that's clear in our docs, and to provide some documentation on how to use cert-manager alongside Garden, reference the generated secrets etc.
FYI @MarkMurphy
FYI @MarkMurphy
Yeah, just went through that process actually.
This issue has been automatically marked as stale because it hasn't had any activity in 90 days. It will be closed in 14 days if no further activity occurs (e.g. changing labels, comments, commits, etc.). Please feel free to tag a maintainer and ask them to remove the label if you think it doesn't apply. Thank you for submitting this issue and helping make Garden a better product!
Maybe a first step on the path to deprecation would be to fix this:
Failed resolving provider kubernetes. Here is the output:
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Error validating provider configuration: key .certManager.email is required
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Failed resolving one or more providers:
- kubernetes
This happens when you set certManager.install: false
This issue has been automatically marked as stale because it hasn't had any activity in 90 days. It will be closed in 14 days if no further activity occurs (e.g. changing labels, comments, commits, etc.). Please feel free to tag a maintainer and ask them to remove the label if you think it doesn't apply. Thank you for submitting this issue and helping make Garden a better product!
Probably not worth solving if certmanager is going to be removed anyway. Stalebot can close it.
This issue has been automatically marked as stale because it hasn't had any activity in 90 days. It will be closed in 14 days if no further activity occurs (e.g. changing labels, comments, commits, etc.). Please feel free to tag a maintainer and ask them to remove the label if you think it doesn't apply. Thank you for submitting this issue and helping make Garden a better product!
Deprecated in #3517 Fully deleted in #4085 Closing this issue as completed.