-
Notifications
You must be signed in to change notification settings - Fork 8
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
CP-20221: Labels and Annotations #127
Conversation
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.
Question: cert-manager is defaulted to enabled, and yet you state that most environments will already have this deployed. is it just me or is this contradictory?
Great question! I think in most "production" environments we've seen they already have a deployment. I think for easy of install we had this enabled, but maybe we should disable it? @dmepham would like your input here. |
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.
Just a few editorial suggestions, but otherwise looks good!
- [`cert-manager` Documentation](https://cert-manager.io/docs/) | ||
- [CloudZero Agent Helm Chart README](./README.md) | ||
- [ValidatingWebhookConfiguration Reference](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.23/#validatingwebhookconfiguration-v1-admissionregistration-k8s-io) | ||
- [Helm Troubleshooting Guide](https://helm.sh/docs/chart_best_practices/troubleshooting/) |
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.
This URL is broken; should we link to https://helm.sh/docs/faq/troubleshooting/
instead?
Co-authored-by: Becki Lee <[email protected]>
Co-authored-by: Becki Lee <[email protected]>
Description
Testing
N/A