Skip to content

[202511][FRR][Patch]bgpd: Show all advertised paths including non best paths only if addpath is enabled#26869

Open
GaladrielZhao wants to merge 1 commit intosonic-net:202511from
GaladrielZhao:fix/202511_vty_adv_routes
Open

[202511][FRR][Patch]bgpd: Show all advertised paths including non best paths only if addpath is enabled#26869
GaladrielZhao wants to merge 1 commit intosonic-net:202511from
GaladrielZhao:fix/202511_vty_adv_routes

Conversation

@GaladrielZhao
Copy link
Copy Markdown
Contributor

Why I did it

Fix issue #26739:
Bug: vtysh show ip bgp neighbor advertised-routes output all

How I did it

Port from upstream FRR PR FRRouting/frr#20618:
bgpd: Show all advertised paths including non-best paths only if addpath is enabled

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

A picture of a cute animal (not mandatory but encouraged)

image

…only if addpath is enabled

Fix issue sonic-net#26739:
Bug: vtysh show ip bgp neighbor advertised-routes output all

From FRR PR FRRouting/frr#20618:
bgpd: Show all advertised paths including non-best paths only if addpath is enabled

Signed-off-by: Yuqing Zhao <[email protected]>
@GaladrielZhao GaladrielZhao requested a review from lguohan as a code owner April 17, 2026 08:24
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao
Copy link
Copy Markdown
Contributor Author

GaladrielZhao commented Apr 18, 2026

Hi @ton31337 , could you please review this patch?
This is the 202511 branch version of the patch in PR sonic-net#26870, and is also ported from PR FRRouting/frr#20618.
The logic to resolve the conflict in route_vty_out() is aligned with PR sonic-net#26870, using the code base of frr-10.4.1.
Thanks a lot.

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