-
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
l2announcer: Fix delete entry if no origins left #32279
Conversation
Commit 7865454 does not match "(?m)^Signed-off-by:". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
7865454
to
d956407
Compare
Commit 11ab162 does not match "(?m)^Signed-off-by:". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
When enabling L2Announcer and using the sharedingkeys feature, deleting a service that uses the same requestIP will unexpectedly delete the allocated LB IP in the Cilium agent. Fixes: cilium#32115 Signed-off-by: Taizeng Wu <wutaizeng@gmail.com>
11ab162
to
8a0362e
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.
Nice catch! Thanks
When enabling L2Announcer and using the sharedingkeys feature, deleting a service that uses the same requestIP will unexpectedly delete the allocated LB IP in the Cilium agent.
Fixes: #32115