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.

Sign up for GitHub

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

Jump to bottom

Discover the path MTU to be used in OVN route agent driver. #3097

Open
aswinsuryan opened this issue Jul 22, 2024 · 1 comment
Open

Discover the path MTU to be used in OVN route agent driver. #3097

aswinsuryan opened this issue Jul 22, 2024 · 1 comment
Assignees
@yboaron
Labels
enhancement New feature or request

Comments

@aswinsuryan
Copy link
Contributor

aswinsuryan commented Jul 22, 2024 •
edited
Loading

We use hardcode MTU values in gateway_dataplane.go.

This needs to be enhanced to use the real path MTU calculated using tools like tracepath.

This is required as described here #1022

What would you like to be added:

Why is this needed:
@aswinsuryan aswinsuryan added the enhancement New feature or request label Jul 22, 2024
@aswinsuryan aswinsuryan mentioned this issue Jul 22, 2024
Merged
aswinsuryan added a commit to aswinsuryan/submariner that referenced this issue Jul 22, 2024
@aswinsuryan
Remove TODO since an issues is raised submariner-io#3097
e58bf23 
Signed-off-by: Aswin Suryanarayanan <[email protected]>
@yboaron yboaron self-assigned this Jul 23, 2024
@yboaron
Copy link
Contributor

yboaron commented Jul 23, 2024

I guess we should retry checking with mss clamping rule to pmtu first

skitt pushed a commit that referenced this issue Jul 23, 2024
@aswinsuryan @skitt
Remove TODO since an issues is raised #3097
3bb73cd 
Signed-off-by: Aswin Suryanarayanan <[email protected]>
aswinsuryan added a commit to aswinsuryan/submariner that referenced this issue Jul 23, 2024
@aswinsuryan
Remove TODO in ../ovn/gateway_dataplane.go since an issue is raised s…
5fd063d 
…ubmariner-io#3097

Signed-off-by: Aswin Suryanarayanan <[email protected]>
@aswinsuryan aswinsuryan mentioned this issue Jul 23, 2024
Merged
aswinsuryan added a commit to aswinsuryan/submariner that referenced this issue Jul 23, 2024
@aswinsuryan
Remove TODO ../ovn/gateway_dataplane.go since an issue is raised subm…
f839975 
…ariner-io#3097

Signed-off-by: Aswin Suryanarayanan <[email protected]>
tpantelis pushed a commit that referenced this issue Jul 23, 2024
@aswinsuryan @tpantelis
Remove TODO ../ovn/gateway_dataplane.go since an issue is raised #3097
ae618af 
Signed-off-by: Aswin Suryanarayanan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yboaron yboaron

Labels
enhancement New feature or request
Projects
Submariner 0.19
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aswinsuryan @yboaron