Skip to content

Conversation

ipspace
Copy link
Owner

@ipspace ipspace commented Sep 30, 2025

Using the infrastructure created in #2656, the VRF module validation of the loopback interface data allows module-specific loopback parameters.

The extra loopback VRF parameters are copied into the interface data when the VRF module creates VRF loopback interfaces in the node data.

Also:

  • Add 'bgp.advertise' as a valid loopback parameter as we have no VRF equivalent of 'bgp.advertise_loopback' node parameter.
  • Add VRF loopback OSPF area to a transformation test
  • Create a new error test that checks the VRF loopback validation code

Implements #2659

Using the infrastructure created in #2656, the VRF module validation of
the loopback interface data allows module-specific loopback parameters.

The extra loopback VRF parameters are copied into the interface data
when the VRF module creates VRF loopback interfaces in the node data.

Also:
* Add 'bgp.advertise' as a valid loopback parameter as we have no
  VRF equivalent of 'bgp.advertise_loopback' node parameter.
* Add VRF loopback OSPF area to a transformation test
* Create a new error test that checks the VRF loopback validation code

Implements #2659
@ipspace ipspace requested a review from jbemmel September 30, 2025 10:31
@ipspace ipspace mentioned this pull request Oct 4, 2025
Copy link
Collaborator

@jbemmel jbemmel left a comment

Choose a reason for hiding this comment

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

Couple of minor items

@ipspace
Copy link
Owner Author

ipspace commented Oct 5, 2025

Couple of minor items

Thank you! Fixed.

@ipspace ipspace merged commit 2d73039 into dev Oct 5, 2025
12 checks passed
@ipspace ipspace deleted the vrf-lb-attr branch October 5, 2025 08:38
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