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

Fix for MTU change: get L3 mtu instead of linkMTU #555

Merged

Conversation

ljkiraly
Copy link
Contributor

Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: Laszlo Kiraly laszlo.kiraly@est.tech

Issue: networkservicemesh/cmd-forwarder#557
Related PR: networkservicemesh#554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: Laszlo Kiraly <laszlo.kiraly@est.tech>
@ljkiraly ljkiraly marked this pull request as ready for review April 12, 2022 08:22
@edwarnicke edwarnicke merged commit 19f2f16 into networkservicemesh:main Apr 12, 2022
nsmbot pushed a commit to networkservicemesh/cmd-nse-vl3-vpp that referenced this pull request Apr 12, 2022
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#555

Commit: 19f2f16
Author: Laszlo Kiraly
Date: 2022-04-12 17:13:09 +0200
Message:
  - Fix for MTU change: get L3 mtu instead of linkMTU (#555)
Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-firewall-vpp that referenced this pull request Apr 12, 2022
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#555

Commit: 19f2f16
Author: Laszlo Kiraly
Date: 2022-04-12 17:13:09 +0200
Message:
  - Fix for MTU change: get L3 mtu instead of linkMTU (#555)
Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-icmp-responder-vpp that referenced this pull request Apr 12, 2022
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#555

Commit: 19f2f16
Author: Laszlo Kiraly
Date: 2022-04-12 17:13:09 +0200
Message:
  - Fix for MTU change: get L3 mtu instead of linkMTU (#555)
Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-vpp that referenced this pull request Apr 12, 2022
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#555

Commit: 19f2f16
Author: Laszlo Kiraly
Date: 2022-04-12 17:13:09 +0200
Message:
  - Fix for MTU change: get L3 mtu instead of linkMTU (#555)
Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-forwarder-vpp that referenced this pull request Apr 12, 2022
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#555

Commit: 19f2f16
Author: Laszlo Kiraly
Date: 2022-04-12 17:13:09 +0200
Message:
  - Fix for MTU change: get L3 mtu instead of linkMTU (#555)
Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vlan-vpp that referenced this pull request Apr 12, 2022
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#555

Commit: 19f2f16
Author: Laszlo Kiraly
Date: 2022-04-12 17:13:09 +0200
Message:
  - Fix for MTU change: get L3 mtu instead of linkMTU (#555)
Issue: networkservicemesh/cmd-forwarder#557
Related PR: #554

The L3 MTU is set based on link L2 mtu and should not modified.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants