Skip to content

fpmsyncd : Changes for supporting install-then-advertise behavior#1101

Open
sudhanshukumar22 wants to merge 2 commits intosonic-net:masterfrom
sudhanshukumar22:bgp-install-advertise-feature-sonic
Open

fpmsyncd : Changes for supporting install-then-advertise behavior#1101
sudhanshukumar22 wants to merge 2 commits intosonic-net:masterfrom
sudhanshukumar22:bgp-install-advertise-feature-sonic

Conversation

@sudhanshukumar22
Copy link

fpmsyncd : Changes for supporting install-then-advertise behavior
Functional specification:
sonic-net/SONiC#424
This PR is dependent on the following PR and full-build can fail since the changes in the below PR (swss-common) are needed.
sonic-net/sonic-swss-common#309

The routes will be pushed by BGP to Zebra to fpm to be installed
in hardware. If fpm returns error, the routes will not be sent by BGP to
its peers. Only successful routes are sent by BGP to its peers. The
feature can be enabled/disabled in BGP by a CLI. By default, the feature
is disabled. Functional specification:
sonic-net/SONiC#424
Signed-off by: Preetham Singh [email protected]
Signed-off by: Sudhanshu Kumar [email protected]

What I did

Why I did it

How I verified it

Details if related

sudhanshukumar22 and others added 2 commits October 18, 2019 01:13
The  routes will be pushed by BGP to Zebra to   fpm to be installed
in  hardware.  If fpm returns error, the routes will not  be sent by BGP to
its  peers. Only  successful routes are sent by BGP to its  peers. The
feature can  be  enabled/disabled in BGP by a CLI. By default, the  feature
is disabled.  Functional specification:
sonic-net/SONiC#424
Signed-off by: Preetham Singh <[email protected]>
Signed-off by: Sudhanshu Kumar <[email protected]>
@prsunny prsunny self-requested a review as a code owner September 2, 2022 23:17
jianyuewu pushed a commit to jianyuewu/sonic-swss that referenced this pull request Dec 24, 2025
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.

1 participant