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

Need new test: BGP-Convergence #3608

Closed
kevinskwang opened this issue Jun 4, 2021 · 4 comments
Closed

Need new test: BGP-Convergence #3608

kevinskwang opened this issue Jun 4, 2021 · 4 comments

Comments

@kevinskwang
Copy link
Collaborator

kevinskwang commented Jun 4, 2021

This is a test gap:

Need to add BGP convergence test.

@yxieca
Copy link
Collaborator

yxieca commented Oct 21, 2021

Convergence: Route should be program first in FIB and then advertised

@yxieca yxieca added the P1 label Oct 21, 2021
@kamalsahu0001
Copy link
Contributor

Keysight will be taking care of this test case. We already have the RIB-IN Convergence test covered in BGP Convergence test plan. In our case, we are verifying that initially there won't be any traffic flowing and once we start advertising the routes, we are measuring the time it takes to install in FIB and then to forward the traffic and no traffic loss once all the routes are advertised. Will this verification steps satisfies the test case? or we need any additional verification steps?

@StormLiangMS StormLiangMS self-assigned this Jun 4, 2024
@wsycqyz wsycqyz assigned wsycqyz and unassigned StormLiangMS Jun 20, 2024
@wsycqyz
Copy link
Contributor

wsycqyz commented Jul 31, 2024

Currently there is testplan:
docs/testplan/BGP Convergence Testplan for single DUT.md
docs/testplan/BGP-Convergence-Testplan-for-Benchmark-Performance.md.

there is testcase:
tests/snappi_tests/bgp/test_bgp_convergence_performance.py
tests/snappi_tests/bgp/test_bgp_rib_in_convergence.py

I have read these documents and found the current tests cover the most common scenarios.

@StormLiangMS Let's discuss offline to see if there is any gap here.
@yxieca FYI.

@wsycqyz
Copy link
Contributor

wsycqyz commented Aug 6, 2024

Following PR has been merged for this test gap. So close this test gap as finished.
PR #2926
PR #3715
PR #5098

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

5 participants