-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
docs: Updates k8s annotation docs #13809
Conversation
Updates k8s annotation docs for the options added in hashicorp/consul-k8s#1315.
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.
Overall LGTM to me with these minor suggestions.
Please let me know if you have any questions or need any further assistance!
Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com>
I added some more detail here about how to populate the actual values for this annotation, as it is not very apparent on how to do based on the description that was provided. |
* Updates k8s annotation docs Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com> Co-authored-by: David Yu <dyu@hashicorp.com>
Updates k8s annotation docs for the options added in hashicorp/consul-k8s#1315.
Description
Describe why you're making this change, in plain English.
Testing & Reproduction steps
Links
Include any links here that might be helpful for people reviewing your PR (Tickets, GH issues, API docs, external benchmarks, tools docs, etc). If there are none, feel free to delete this section.
Please be mindful not to leak any customer or confidential information. HashiCorp employees may want to use our internal URL shortener to obfuscate links.
PR Checklist