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: FRR Changes for supporting bgp route install-then-advertise behavior #3623

Conversation

sudhanshukumar22
Copy link
Contributor

This contains the FRR changes for supporting BGP route install-then-advertise behavior.
Functional specification:
sonic-net/SONiC#424

When BGP learns a route from its peer, the routes will be pushed by BGP to Zebra to fpm to be installed in hardware. If fpm returns an 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.
Signed-off by: Preetham Singh [email protected]

- What I did

- How I did it

- How to verify it

- Description for the changelog

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

@pavel-shirshov pavel-shirshov self-assigned this Oct 17, 2019
@pavel-shirshov pavel-shirshov self-requested a review October 17, 2019 21:22
@sudhanshukumar22 sudhanshukumar22 force-pushed the bgp-install-advertise-feature branch from d982f81 to bdf7d24 Compare October 18, 2019 10:04
mssonicbld added a commit that referenced this pull request Dec 15, 2024
…atically (#21161)

#### Why I did it
src/sonic-utilities
```
* 200ef363 - (HEAD -> master, origin/master, origin/HEAD) Speed up route_check script (#3678) (32 hours ago) [Deepak Singhal]
* 7dc40ac3 - Fixed the issues with sonic-clear queuecounter for egress queue and voq (#3671) (2 days ago) [saksarav-nokia]
* 72ee4fc1 - [config db] Trim garbage charactor in "DEVICE_METADATA" of config db (#3345) (2 days ago) [wenyiz2021]
* b2ba0825 - [show][interfaces] Add proposal for show interface errors {port} (#3623) (3 days ago) [vdahiya12]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

3 participants