-
Notifications
You must be signed in to change notification settings - Fork 737
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
BGP Convergence Performance and Scalability Cases #5098
Conversation
Get latest changes
This pull request introduces 4 alerts when merging 2d56f9d into 1a6806c - view on LGTM.com new alerts:
|
This pull request introduces 4 alerts when merging 79092ff into 3acec36 - view on LGTM.com new alerts:
|
/azpw run |
Commenter does not have sufficient privileges for PR 5098 in repo Azure/sonic-mgmt |
This pull request introduces 1 alert when merging 7d33881 into fe8fee6 - view on LGTM.com new alerts:
|
@wangxin can you please approve this PR |
@wangxin ================================================================================== warnings summary =================================================================================== snappi/bgp/test_bgp_convergence_performance.py::test_bgp_convergence_performance[IPv4-16500-500-500-2] -- Docs: https://docs.pytest.org/en/latest/warnings.html |
@wangxin here is the log for test_ bgp_scalability.py ----------------------------- live log collection ------------------------------ -------------------------------- live log setup -------------------------------- 20:55:03 connection._warn L0321 WARNING| Verification of certificates is disabled 21:02:06 bgp_test_gap_helper.restart_traffic L0712 INFO | L2/3 traffic apply failed,Restarting protocols and traffic 21:02:09 reboot.reboot L0133 INFO | waiting for ssh to drop on sonic-s6100-dut 21:16:52 bgp_test_gap_helper.restart_traffic L0712 INFO | L2/3 traffic apply failed,Restarting protocols and traffic 21:16:55 reboot.reboot L0133 INFO | waiting for ssh to drop on sonic-s6100-dut =============================== warnings summary =============================== snappi/bgp/test_bgp_scalability.py::test_bgp_scalability_16k_v4_routes[16000-1] -- Docs: https://docs.pytest.org/en/latest/warnings.html |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@wangxin , Getting flake8 failures during azp run, the imported modules are required for the script to run. in test_bgp_convergence_performance.py cvg_api is used in line 16, similarly for other modules as well. These look like false failure |
@wangxin can you please approve this PR |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
BGP Convergence Performance and Scalability Cases BGP Convergence Performance BGP Scalability
@selldinesh PR conflicts with 202305 branch |
@developfast This is PR is to add new tests. Do we really need it in 202205 branch? |
@wangxin yes. |
@selldinesh PR conflicts with 202205 branch |
Description of PR
Summary:BGP Convergence Performance and Scalability Cases
BGP Convergence Performance
BGP Scalability
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
How did you do it?
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation
https://github.com//issues/3609#3610