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 Issues 239 & 262 #263

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from
Draft

Fix for Issues 239 & 262 #263

wants to merge 7 commits into from

Conversation

mtarking
Copy link
Collaborator

@mtarking mtarking commented Dec 20, 2024

Related Issue(s)

Fix to resolve #262
Also, this aims to resolve #239

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.global
  • vxlan.topology
  • vxlan.underlay
  • vxlan.overlay_services
  • vxlan.overlay_extensions
  • vxlan.policy
  • defaults.vxlan
  • other

Proposed Changes

Remove unintended IPv6 check wrapper.
Update templating to better support IPv6 underlay scenarios.

Test Notes

Different permutations of IPv4 and IPv6 fabric creation tested with:

  • IPv4 only
  • IPv4 only with TRM
  • IPv6 only with link local
  • IPv6 only without link local
  • IPv6 only with TRM

Cisco NDFC Version

12.2.2

Checklist

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

@mtarking mtarking added the wip work-in-progress label Dec 20, 2024
@mtarking mtarking self-assigned this Dec 20, 2024
@mtarking mtarking changed the title Fix for Issue-262 Fix for Issues 239 & 262 Dec 20, 2024
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
2 participants