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: Improve wording for labels and services policies #27171
Conversation
/test |
68de7c3
to
14ceecf
Compare
/test |
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, pending one quick question. Thanks!
14ceecf
to
e02e00a
Compare
/test |
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!
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.
nits.
PR is in draft, going to remove ready-to-merge label. |
A user pointed out that what we call "Labels-based" policies are actually matching the labels on Endpoints, but also explicitly *not* matching labels on Services. To make this more clear, change the name in the docs to Endpoints based policies. Co-authored-by: Nathan Sweet <nathanjsweet@users.noreply.github.com> Signed-off-by: Joe Stringer <joe@cilium.io>
A user pointed out that the position of this callout made it seem like the callout only applies to the first example rather than applying to all Services based policies. Move the callout to the top of the section to make the relationship clearer. Signed-off-by: Joe Stringer <joe@cilium.io>
03f955d
to
6633dda
Compare
/test |
Based on user feedback, improve the way that this documentation reads in order
to clarify how these policy types apply to Cilium objects.