-
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
daemon: Remove obsolete bpf-lb-dev-ip-addr-inherit option #29963
Conversation
This option was added for a niche use-case that no longer needs it and the agent did not anymore support it. Remove the remaining code related to it. Signed-off-by: Jussi Maki <jussi@isovalent.com>
To give some more background, this was added for a specific use-case for one user of Cilium and it was confirmed that use-case is gone now. We can fairly safely assume this option was never used by anyone else as it was never exposed via the helm values. |
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!
/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 for me
This option was added for a niche use-case that no longer needs it and the agent did not anymore support it. Remove the remaining code related to it.