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
labelsfilter: Ensure entity relevant labels are always applied #31178
Conversation
/test |
Entities are special selectors used by network policies. The Cluster entity relies on the `io.cilium.k8s.policy.cluster` label which is removed by Cilium if a strict identity label configuration is applied. This PR adds the relevant Cilium policy label to the list of default labels so it will always be applied regardless of configuration, and includes this label to the associated test file. Fixes: cilium#18878 Signed-off-by: soggiest <nicholas@isovalent.com>
9361d6e
to
be5a244
Compare
I realized my previous commit contained an extra label that wasn't actually used by as a selector for |
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.
Looks good! My only request would be a slightly more descriptive release note. But, I like it.
/test |
Would someone mind kicking off another test, the errors I'm seeing from the EKS and Ginkgo tests don't seem related to my change. I'm wondering if these are flakes. Tangent question: Why is this network policy file named |
Tangent looks like a bug. May be worth filing an issue and/or PR and/or use |
Entities are special selectors used by network policies. The Cluster entity relies on the
io.cilium.k8s.policy.cluster
label to select endpoints which is removed by Cilium if a strict identity label configuration is applied. This PR adds the relevant Cilium policy label to the list of default labels so it will always be applied regardless of configuration, and includes this label to the associated test.Fixes: #18878
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.