Skip to content

Conversation

@jbemmel
Copy link
Collaborator

@jbemmel jbemmel commented Nov 23, 2024

  • Removes interface type test
  • Puts interfaces in the correct VRF
  • Fixes IPv6 to send out router-advertisements ( "off" means enable them ... )
  • Updates docs (note I could not find bandwidth setting in NVUE)

Note that Cumulus supports multiple loopbacks by putting multiple IPs on a single loopback device; set commands are cumulative and don't replace existing addresses

This also means applying Netlab configs to a system that already has some existing configs won't replace those; this would require an 'unset' command

Copy link
Owner

@ipspace ipspace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for focusing on the NVUE loopbacks. However, I think we might still have a problem if someone configures more than one loopback per VRF (for example, to generate more prefixes)

@ipspace ipspace merged commit 02c4592 into ipspace:dev Nov 24, 2024
5 checks passed
@jbemmel jbemmel deleted the cumulus_nvue_loopback_only branch January 8, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants