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

bgpd: Implement match src-peer ... command #16946

Merged
merged 6 commits into from
Oct 16, 2024

Conversation

ton31337
Copy link
Member

No description provided.

@frrbot frrbot bot added bgp documentation tests Topotests, make check, etc labels Sep 27, 2024
@ton31337 ton31337 marked this pull request as draft September 27, 2024 13:05
This is handy when you need to do source matching e.g. `match src-peer ...`
on outgoing direction with a route-map.

Signed-off-by: Donatas Abraitis <[email protected]>
To match source peer on the outgoing direction.

Signed-off-by: Donatas Abraitis <[email protected]>
Signed-off-by: Donatas Abraitis <[email protected]>
If using at incoming direction, e.g.:

route-map test permit 10
 match src-peer 10.0.0.1
!
router bgp 65000
 neighbor 10.0.0.1 route-map test in
!

Signed-off-by: Donatas Abraitis <[email protected]>
@ton31337 ton31337 marked this pull request as ready for review September 27, 2024 20:05
@riw777 riw777 self-requested a review October 8, 2024 15:47
Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

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

looks good

@riw777
Copy link
Member

riw777 commented Oct 15, 2024

does the one lint need to be taken care of? it's a line wrap thing ... I'm pretty ambivalent about either format, but just want to make certain we don't add new lints

@ton31337
Copy link
Member Author

yeah, it's annoying, but we intentionally keep such wraps 😎

@riw777 riw777 merged commit 80dc863 into FRRouting:master Oct 16, 2024
11 checks passed
@ton31337 ton31337 deleted the fix/match_src-peer branch October 16, 2024 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants