-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: example cluster-wide health endpoint #15348
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.
Thanks a lot for your contribution! This looks good to me on the documentation side, please just find a nitpick below.
Also, sorry to ask, but did you use your real name in the Signed-off-by:
tag? Please note that this cannot be a pseudonym.
@qmonnet Ah, sorry. I will fix it. |
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.
Thanks for the pull request! 🚀
I don't think the policy for the health endpoint is correct. It would probably be worth testing if you haven't already.
859807c
to
c822677
Compare
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.
Thanks for fixing the message and the SOB tag!
One more remark on the new version, below.
Please also note that your changes belong to a single logical commit. This is not an issue to have several commits with your incremental changes for the review, but you'll need to squash them and to force-push on your branch at some point before we merge the PR.
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.
Could you squash the commits into a single one? We should be good to merge after that.
Signed-off-by: Rei Shimizu <Shikugawa@gmail.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.
Thanks a lot!
I don't think we need to wait for the @cilium/policy review as I tested this. Marking as ready to merge. |
Signed-off-by: Shikugawa Shikugawa@gmail.com
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: #14871
This PR adds some descriptions and examples to configure cluster-wide health endpoint.