Skip to content

Conversation

juarocha
Copy link
Contributor

@juarocha juarocha commented Jul 23, 2025

Related Issue(s)

Solves #470

Related Collection Role

  • cisco.nac_dc_vxlan.validate
  • cisco.nac_dc_vxlan.dtc.create
  • cisco.nac_dc_vxlan.dtc.deploy
  • cisco.nac_dc_vxlan.dtc.remove
  • other

Related Data Model Element

  • vxlan.fabric
  • vxlan.global
  • vxlan.topology
  • vxlan.underlay
  • vxlan.overlay
  • vxlan.overlay_extensions
  • vxlan.policy
  • vxlan.multisite
  • defaults.vxlan
  • other

Proposed Changes

Validation script for:

  • Fabric interfaces MTU : accepted value must be an even number in the range 1550 and 9216. minimum should be 1550 due to VXLAN header (Fragmentation is not supported).
  • L2 interfaces MTU: accepted values are 1500, 9216, default(1500) and jumbo(as defined on global value: ). Value must be can be either default or jumbo.
  • L3 interfaces MTU: accepted value must be an even number in the range 1500 and 9216.
  • Global values:
    • layer2_host_interface_mtu: accepted value must be an even number in the range 1550 and 9216 and 'default'(1500)
    • Sub interface MTU must be lower/equal than the parent interface

Test Notes

NA

Cisco NDFC Version

12.2.3

Checklist

  • Latest commit is rebased from develop with merge conflicts resolved
  • New or updates to documentation has been made accordingly
  • Assigned the proper reviewers

@juarocha juarocha marked this pull request as ready for review July 30, 2025 23:41
@juarocha juarocha requested a review from a team as a code owner July 30, 2025 23:41
@juburnet juburnet added the ready for review PR Ready for Review label Jul 31, 2025
@juburnet juburnet requested review from mtarking and ccoueffe July 31, 2025 01:40
@juburnet
Copy link
Collaborator

@juarocha - I approved the sanity checks to run and there were a few errors found. Please review https://github.com/netascode/ansible-dc-vxlan/actions/runs/16636032225/job/47081860486?pr=501 and make the necessary commits with corrections. Let us know if you have any questions.

@juburnet juburnet added ready for review PR Ready for Review and removed ready for review PR Ready for Review labels Jul 31, 2025
@juburnet
Copy link
Collaborator

juburnet commented Aug 1, 2025

All sanity checks are passing now. Ready for review.

@mikewiebe mikewiebe removed the ready for review PR Ready for Review label Aug 8, 2025
@devegupt devegupt added the ready for review PR Ready for Review label Aug 14, 2025
@mikewiebe mikewiebe added wip work-in-progress and removed ready for review PR Ready for Review labels Sep 10, 2025
@juarocha juarocha closed this Sep 10, 2025
@juarocha juarocha force-pushed the schema-mtu_interfaces branch from ef56176 to 268eaf7 Compare September 10, 2025 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wip work-in-progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants