-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
pkg/endpoint: Readd GetRealizedPolicyRuleLabelsForKey #14257
pkg/endpoint: Readd GetRealizedPolicyRuleLabelsForKey #14257
Conversation
This brings back the GetRealizedPolicyRuleLabelsForKey helper which is used by external client code and was accidentally removed as unused. This commit also extends the Hubble unit tests to invoke it, to avoid another accidental removal. Fixes: 0ba1967 ("pkg/endpoint: remove unused functions") Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
test-me-please |
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.
👍 for my code owners (I haven't looked at the new Hubble test in details).
VM provisioning fail on runtime and 4.9: https://jenkins.cilium.io/job/Cilium-PR-Runtime-4.9/2842/ / https://jenkins.cilium.io/job/Cilium-PR-K8s-1.20-kernel-4.9/9/ |
retest-runtime |
retest-4.9 |
FWIW, removing GetRealizedPolicyRuleLabelsForKey breaks network policy logging on GKE too. |
I don't think we can ever remove exported functionality anymore given that Cilium is a dependency for Hubble, GKE network policy logging, etc. |
retest-net-next Edit: VM provisioning fail |
retest-net-next Provisioning keeps failing https://jenkins.cilium.io/job/Cilium-PR-K8s-1.12-net-next/1512/ |
retest-net-next |
All required tests have passed. |
This brings back the GetRealizedPolicyRuleLabelsForKey helper which is
used by external client code and was accidentally removed as unused.
This commit also extends the Hubble unit tests to invoke it, to avoid
another accidental removal.
Fixes: 0ba1967 ("pkg/endpoint: remove unused functions")