Skip to content

Fix bootstrap isn - #473

Merged
mtarking merged 6 commits into
netascode:developfrom
ccoueffe:fix_bootstrap_isn
Jul 12, 2025
Merged

Fix bootstrap isn#473
mtarking merged 6 commits into
netascode:developfrom
ccoueffe:fix_bootstrap_isn

Conversation

@ccoueffe

@ccoueffe ccoueffe commented Jul 12, 2025

Copy link
Copy Markdown
Collaborator

Related Issue(s)

Fix #472

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

  • Update ISN inventory and fabric with new path for bootstrap
  • Update Create Fabric task for ISN to set skip validation.
  • Update default value for bootstrap / isn

Test Notes

Cisco NDFC Version

Checklist

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

ccoueffe added 4 commits July 12, 2025 08:45
update bootstrap path and add condition similar to common/inventory
update path for bootstrap with isn
Add skip_validation: true for ISN. Validation must be skip for ISN with bootstrap.

With validation, you cannot configure bootstrap and you have this kind of message

 IS_READ_ONLY(False) requires BOOTSTRAP_ENABLE != True. BOOTSTRAP_ENABLE valid values: [False, True]. '
update default for bootstrap since key bootstrap for ISN is under multisite.isn
@ccoueffe ccoueffe self-assigned this Jul 12, 2025
@ccoueffe
ccoueffe marked this pull request as ready for review July 12, 2025 07:22
@ccoueffe
ccoueffe requested a review from a team as a code owner July 12, 2025 07:22
@ccoueffe ccoueffe added the ready for review PR Ready for Review label Jul 12, 2025
@ccoueffe
ccoueffe requested a review from mtarking July 12, 2025 07:30
@mtarking
mtarking merged commit 95f0a16 into netascode:develop Jul 12, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review PR Ready for Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Not able to bootstrap switches with ISN fabric

2 participants