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

Advertise routes matching bgp path #17777

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pguibert6WIND
Copy link
Member

bgpd: fix display the attr from adj-rib-out with detail used
The 'show bgp neighbors advertised-routes detail'
command displays all the bgp paths of the loc-rib. The expectation is
that it displays only the bgp path for the peer this path is sent to.

Fix this by replacing the extra attr attribute with the bgp_adj_out
pointer of the path to display. Filter out the appropriate bgp path and
display the adj->attr.

The 'show bgp <afi> <safi> neighbors <peer> advertised-routes detail'
command displays all the bgp paths of the loc-rib. The expectation is
that it displays only the bgp path for the peer this path is sent to.

Fix this by replacing the extra attr attribute with the bgp_adj_out
pointer of the path to display. Filter out the appropriate bgp path and
display the adj->attr.

Fixes: e960b4c ("bgpd: add "detail" for advertised/received-routes")

Signed-off-by: Philippe Guibert <[email protected]>
Add a test that check that the detailed command of show bgp advertised
neighbors 10.125.0.2 displays the locpref value.

Signed-off-by: Philippe Guibert <[email protected]>
@pguibert6WIND pguibert6WIND force-pushed the advertise_routes_matching_bgp_path branch from 572bb5f to b7fd45c Compare January 7, 2025 08:31
@github-actions github-actions bot added size/L and removed size/M labels Jan 7, 2025
@pguibert6WIND pguibert6WIND marked this pull request as draft January 7, 2025 13:53
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant