Skip to content
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

MSS clamping for outgoing TCP packets with OVN #1294

Merged
merged 1 commit into from
Apr 29, 2021

Conversation

mangelajo
Copy link
Contributor

This clamps the maximum segment size on the TCP negotiation
when addressing remote clusters with OVN. With the OVN
implementation we have an issue where the need-frag ICMPs
used for PMTU discovery are sent into the wront interface
by the kernel, this limits the scope of the issue, although
it introduces a performance penalty for jumboframe capable
networks.

Fixes-Issue: #1278

Signed-off-by: Miguel Angel Ajo majopela@redhat.com

This clamps the maximum segment size on the TCP negotiation
when addressing remote clusters with OVN. With the OVN
implementation we have an issue where the need-frag ICMPs
used for PMTU discovery are sent into the wront interface
by the kernel, this limits the scope of the issue, although
it introduces a performance penalty for jumboframe capable
networks.

Fixes-Issue: submariner-io#1278

Signed-off-by: Miguel Angel Ajo <majopela@redhat.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1294/mangelajo/ovn-mss-clamping

@mangelajo mangelajo self-assigned this Apr 28, 2021
@mangelajo mangelajo added 0.9.0-testday blocker A blocker bug bug Something isn't working datapath Datapath related issues or enhancements OVN labels Apr 28, 2021
@mangelajo mangelajo added this to the 0.9.0 milestone Apr 28, 2021
@skitt skitt merged commit 470b148 into submariner-io:devel Apr 29, 2021
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr1294/mangelajo/ovn-mss-clamping]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.9.0-testday blocker A blocker bug bug Something isn't working datapath Datapath related issues or enhancements OVN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants