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
CNI: add host-side interface info to cni.Result #26518
Conversation
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.
One minor request, then looks good!
Off topic and unrelated to the fix, have you considered using Cilium's bandwidth manager? https://docs.cilium.io/en/stable/network/kubernetes/bandwidth-manager/ |
mirroring +1 from last pr which was closed: #24901 |
/test |
Issue #24899 which this PR fixes mentions "we cannot use cilium bandwidthManager because our os kernel version is too low to meet the requirements.". |
This commit adds host-side interface name and mac address to cni.Result according to CNI Specification (https://www.cni.dev/docs/spec/#section-5-result-types) Signed-off-by: czybjtu <smartczy@outlook.com>
e84b7c4
to
bd3ef30
Compare
/test |
Marking for backport since this is a bug fix. |
Fixes: #24899