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

Inconsistent TCP MSS Ceiling Schema #4620

Open
1 task done
perimore opened this issue Oct 21, 2024 · 1 comment
Open
1 task done

Inconsistent TCP MSS Ceiling Schema #4620

perimore opened this issue Oct 21, 2024 · 1 comment
Assignees
Labels
type: bug Something isn't working

Comments

@perimore
Copy link
Contributor

perimore commented Oct 21, 2024

Issue Summary

Inconsistency between ethernet_interfaces and tunnel_interfaces when configuring TCP MSS ceiling segment size.

ethernet_interfaces uses the below:

            ethernet_interfaces:
              - name: Ethernet10
                tcp_mss_ceiling:
                  ipv4_segment_size: 1500

Whereas tunnel_interfaces uses:

            tunnel_interfaces:
              - name: Tunnel1
                tcp_mss_ceiling:
                  ipv4: 1500

Rendering the same config:

interface Ethernet10
   switchport
   tcp mss ceiling ipv4 1500
!
interface Tunnel1
   tcp mss ceiling ipv4 1500

We would expect them both to use the same key. Considering support in ethernet_interfaces was implemented first, I would suggest using ipv4_segment_size

Which component(s) of AVD impacted

eos_cli_config_gen

How do you run AVD ?

PyAVD

Steps to reproduce

No response

Relevant log output

No response

Contributing Guide

  • I agree to follow this project's Code of Conduct
@perimore perimore added the type: bug Something isn't working label Oct 21, 2024
@gmuloc
Copy link
Contributor

gmuloc commented Oct 28, 2024

This model is used in 4 places:

  • ethernet_interfaces
  • tunnel_interfaces
  • dps_interfaces
  • router_path_selection

two are using ipv4 / ipv6 and two are using ipv4_segment_size / ipv6_segment_size.

Following AVD spirit - we will keep ipv4 / ipv6.
The two places with ipvX_segment_size (namely ethernet_interfaces and router_path_selection) need to be added the new keys ipv4 and ipv6. The ipvX_segment_size keys will be deprecated and set to be removed in AVD 6.0.

eos_designs output needs to be updated to use the newer keys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants