-
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
daemon/cmd: update visibility policy after endpoint queue is created #10608
Conversation
Release note label not set, please set the appropriate release note. |
4 similar comments
Release note label not set, please set the appropriate release note. |
Release note label not set, please set the appropriate release note. |
Release note label not set, please set the appropriate release note. |
Release note label not set, please set the appropriate release note. |
test-me-please previous flake https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/17976/ |
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.
Just one question below on potentially queueing multiple regenerations. I'm concerned this may affect all endpoints, not just endpoints with the annotations.
EDIT: Github did weird things. I mean the above comment:
#10608 (review)
test-me-please |
As the endpoint queue is created when adding the endpoint to the endpoint manager we should only update the visibility policy after the endpoint is added to the endpoint manager to avoid seeing error logs every time a new endpoint is created. Fixes: acbf880 ("pkg/endpoint: decrease direct iteractions with k8s-apiserver") Signed-off-by: André Martins <andre@cilium.io>
test-me-please |
As the endpoint queue is created when adding the endpoint to the
endpoint manager we should only update the visibility policy after the
endpoint is added to the endpoint manager to avoid seeing error logs
every time a new endpoint is created.
Fixes: acbf880 ("pkg/endpoint: decrease direct iteractions with k8s-apiserver")
Signed-off-by: André Martins andre@cilium.io
Fixes #10543
This change is