-
Notifications
You must be signed in to change notification settings - Fork 136
[cni-cilium] Fix mtu #16751
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-cilium] Fix mtu #16751
Conversation
Signed-off-by: Andrey Pavlov <andrey.pavlov@flant.com>
This reverts commit 26b16be. Signed-off-by: Andrey Pavlov <andrey.pavlov@flant.com>
24ad614 to
acf07bc
Compare
|
🟢 Workflow detailsStatic-Static-Containerd-1.32 - Connection string: 🟢 |
|
🟢 Workflow detailsVCD-Standard-Containerd-1.32 - Connection string: 🟢 |
Signed-off-by: Andrey Pavlov <andrey.pavlov@flant.com>
|
Cherry pick PR 16801 to the branch release-1.73 successful! |
|
/backport 1.74 |
Signed-off-by: Andrey Pavlov <andrey.pavlov@flant.com>
|
Cherry pick PR 16803 to the branch release-1.74 successful! |
Description
The basic logic of MTU configuration in Cilium is:
However, this logic does not work well in the context of virtualization, where internal interfaces are created with the MTU of the pod and there is no way to determine the MTU of routes. To work correctly with virtualization, we have previously made changes to create pod interfaces with a reduced MTU.
With the latest version of Cilium, a mechanism for automatically updating MTUs on all pod interfaces has been added, while still following its basic logic.
In this PR, we will make the same change to the automatic MTU update mechanism.
Why do we need it, and what problem does it solve?
When using custom high MTU values, different calculation methods in two different places can lead to problems.
Why do we need it in the patch release (if we do)?
This has been identified in production clusters, and it has become a problem.
Checklist
Changelog entries