Skip to content

fix: secondary IP blocks on "external" subnets (SVIs) #792

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 1, 2025
Merged

fix: secondary IP blocks on "external" subnets (SVIs) #792

merged 1 commit into from
Apr 1, 2025

Conversation

@stevekeay
Copy link
Contributor

@stevekeay stevekeay commented Mar 21, 2025 •
edited
Loading

The Nautobot update is supposed to be adding to the set of subnets that are associated with the UCVNI. It was incorrectly replacing the entire set.

With these changes, the network/UCVNI gets correctly associated with both subnets/prefixes:

image

and the correct configuration ends up on the switch:

interface Vlan1100
  description 1100_OUTSIDE___cccbb055
  no shutdown
  ip access-group customer-12200-CUSTOM in
  vrf member PUBLIC
  no ip redirects
  ip address 10.4.88.33/28 tag 50000
  ip address 10.4.88.49/28 secondary tag 50000
  no ipv6 redirects
  fabric forwarding mode anycast-gateway

@stevekeay stevekeay changed the title Fix secondary IP blocks on "external" subnets (SVIs) fix: secondary IP blocks on "external" subnets (SVIs) Mar 21, 2025
@stevekeay stevekeay force-pushed the secondary-ip-blocks branch 2 times, most recently from f592322 to b07486a Compare March 27, 2025 11:38
@stevekeay stevekeay force-pushed the secondary-ip-blocks branch from db521a4 to 340492a Compare April 1, 2025 13:19
@stevekeay
Fix secondary IP blocks on "external" subnets (SVIs)
1429990 
The Nautobot update is supposed to be adding to the set of subnets that
are associated with the UCVNI.  It was incorrectly replacing the entire
set.
@stevekeay stevekeay force-pushed the secondary-ip-blocks branch from 340492a to 1429990 Compare April 1, 2025 14:00
@cardoe cardoe added this pull request to the merge queue Apr 1, 2025
Merged via the queue into main with commit 9a04dcb Apr 1, 2025
29 checks passed
@cardoe cardoe deleted the secondary-ip-blocks branch April 1, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cardoe cardoe cardoe approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stevekeay @cardoe