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: cleanup SPIRE & Envoy values in helm reference #26039
docs: cleanup SPIRE & Envoy values in helm reference #26039
Conversation
This commit cleans up the helm reference documentation of the SPIRE related values, by removing the comments of "objects" from the documentation and replacing them by commenting the properties of the objects if not already present. Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
5cce86d
to
61fac2a
Compare
The helm value properties related to the Envoy DaemonSet doesn't use the proper comment syntax which is the reason why the properties doesn't show up in the documentation. This commit fixes this by using the proper syntax starting with `# --`. Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
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 ✔️
removed review-request for @tommyp1ckles as sig-k8s is already covered by @youngnick & @sayboras |
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
documentation changes only. marking as ready-to-merge |
This PR cleans up the helm reference documentation of the SPIRE & Envoy related values, by removing the comments of "objects" from the documentation and replacing them with comments on the object properties if not already present.
This removes objects like the following from the docu