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

Ebgp unnumbered enhancements #362

Merged
merged 8 commits into from
Aug 28, 2022
Merged

Ebgp unnumbered enhancements #362

merged 8 commits into from
Aug 28, 2022

Conversation

jbemmel
Copy link
Collaborator

@jbemmel jbemmel commented Aug 27, 2022

This example is pushing the limits of what one might setup in a test topology, I wanted to see if I could do EBGP peering locally to leak routes between VRFs.

To make this work, I had to extend neighbor lists with module specific interface attributes ( bgp.underlay_as in this case )

@jbemmel
Copy link
Collaborator Author

jbemmel commented Aug 27, 2022

I tried to keep the module attribute copying generic; the 'vrf' module attribute isn't strictly needed, but it gets included as the vrf module defines an interface level attribute

Alternatively, we could introduce a 'neighbor_interface' category of attributes, containing the list of attributes to include in neighbor lists

bgp:
 attributes:
  neighbor_interface: [ local_as ] # extended with underlay_as in ebgp-local_as plugin

@ipspace ipspace merged commit 9a56812 into ipspace:dev Aug 28, 2022
@ipspace
Copy link
Owner

ipspace commented Aug 28, 2022

Alternatively, we could introduce a 'neighbor_interface' category of attributes, containing the list of attributes to include in neighbor lists

No worries, it's perfectly OK to have remote interface attributes available in the neighbor list. Might come useful in other scenarios as well (example: check OSPF areas).

ipspace pushed a commit that referenced this pull request Aug 28, 2022
ipspace pushed a commit that referenced this pull request Aug 28, 2022
ipspace pushed a commit that referenced this pull request Sep 8, 2022
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