Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Revert "Revert suppress FIB pending feature" #19836

Merged

Conversation

dgsudharsan
Copy link
Collaborator

Reverts #19027

The support for BGP zebra enhancements is merged in #19717. Hence re-enabling BGP suppress fib pending feature.

@dgsudharsan
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liushilongbuaa
Copy link
Contributor

/azpw ms_conflict

@StormLiangMS StormLiangMS requested a review from lipxu August 14, 2024 07:40
@StormLiangMS
Copy link
Contributor

StormLiangMS commented Aug 14, 2024

hi @lipxu could you help to take a look? Or suggest which test we should have nVidia to run to take in this PR?

@dgsudharsan
Copy link
Collaborator Author

hi @lipxu could you help to take a look? Or suggest which test we should have nVidia to run to take in this PR?

Hi @StormLiangMS . We have run the entire sonic-mgmt test suite with these changes + zebra bgp enhancements. We found no issues. We also find memory doesn't increase with test_stress_route with 50 iterations.

Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit 0b4c3a1 into sonic-net:master Sep 2, 2024
23 checks passed
vvolam pushed a commit to vvolam/sonic-buildimage that referenced this pull request Sep 12, 2024
…-net#19836)

Reverts sonic-net#19027

The support for BGP zebra enhancements is merged in sonic-net#19717. Hence re-enabling BGP suppress fib pending feature.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants