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

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

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

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
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