Skip to content

Added support for VPC pairing on the external fabrics - #464

Merged
mtarking merged 15 commits into
developfrom
fix_407_vpc_ext
Sep 10, 2025
Merged

Added support for VPC pairing on the external fabrics#464
mtarking merged 15 commits into
developfrom
fix_407_vpc_ext

Conversation

@peter8498

Copy link
Copy Markdown
Collaborator

Related Issue(s)

https://github.com/netascode/ansible-dc-vxlan/issues/407

Related Collection Role

  • cisco.nac_dc_vxlan.validate
  • [ x] cisco.nac_dc_vxlan.dtc.create
  • cisco.nac_dc_vxlan.dtc.deploy
  • [ x] cisco.nac_dc_vxlan.dtc.remove
  • other

Related Data Model Element

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

Proposed Changes

Adding support for the VPC pairing in the external fabric that requires multiple new input paramaters

Test Notes

Completed testing on an external fabric with a broad sample of inputs:

vxlan:
topology:
vpc_peers:
- peer1: SVS-DC-SL1
peer2: SVS-DC-SL2
enabled: true
trunk_allowed_vlans: all
domain_id: 11
keepalive_vrfname: management
port_channel_mode: active
peer1_keepalive_ipv4: 10.1.150.100
peer2_keepalive_ipv4: 10.1.150.101
peer1_po_channel_desc: "SVS-DC-SL1 to SVS-DC-SL2"
peer2_po_channel_desc: "SVS-DC-SL2 to SVS-DC-SL1"
peer1_member_interfaces: "eth1/15,eth1/16"
peer2_member_interfaces: "eth1/15,eth1/16"

Cisco NDFC Version

12.2.2

Checklist

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

@peter8498
peter8498 requested review from mikewiebe and mtarking July 11, 2025 12:53
@peter8498
peter8498 requested a review from a team as a code owner July 11, 2025 12:53
@peter8498 peter8498 added the ready for review PR Ready for Review label Jul 11, 2025
@peter8498
peter8498 requested a review from juburnet July 11, 2025 13:47
Comment thread roles/dtc/common/tasks/external/ndfc_vpc_peering_pairs.yml Outdated
@mikewiebe mikewiebe removed the ready for review PR Ready for Review label Jul 25, 2025
@peter8498

Copy link
Copy Markdown
Collaborator Author

Updated date and removing unneeded fields in the data model related to vpc plus and vteps

@peter8498
peter8498 requested a review from mikewiebe July 31, 2025 12:59
@peter8498 peter8498 added the ready for review PR Ready for Review label Aug 1, 2025
Updated Jinja references for port channel and fixed defaults
@mikewiebe mikewiebe added the 0.5.0 Release 0.5.0 label Aug 21, 2025
Comment thread roles/dtc/common/templates/ndfc_vpc/ndfc_vpc_peering_pairs_external.j2 Outdated
Comment thread roles/dtc/common/templates/ndfc_vpc/ndfc_vpc_peering_pairs_external.j2 Outdated
Updated after Matt Tarkington's review to adjust namings
@mtarking
mtarking merged commit 53a3bbf into develop Sep 10, 2025
8 checks passed
@mtarking
mtarking deleted the fix_407_vpc_ext branch September 10, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.5.0 Release 0.5.0 ready for review PR Ready for Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants