-
Notifications
You must be signed in to change notification settings - Fork 717
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
[Reclaiming buffer] Test cases for reclaiming buffer on both traditional and dynamic model #4561
[Reclaiming buffer] Test cases for reclaiming buffer on both traditional and dynamic model #4561
Conversation
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
…is defined Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Remove redundant code
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
…e-dynamic-buffer.py Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
…he reference Signed-off-by: Stephen Sun <stephens@nvidia.com>
This pull request introduces 1 alert when merging 3959906 into 001c934 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 8852a69 into 4f52b35 - view on LGTM.com new alerts:
|
Signed-off-by: Stephen Sun <stephens@nvidia.com>
This pull request introduces 1 alert when merging 8d8b5b3 into 46ef49d - view on LGTM.com new alerts:
|
Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs could you please refer which is the PR that this test depends on? For 201911 do we need a seperate PR or this can be cherry picked? |
This is for 202012 and above. For 201911 we already have a or which has been merged. |
…nal and dynamic model (sonic-net#4561) ### Description of PR Summary: Test cases for reclaiming buffer on both traditional and dynamic model ### Approach #### What is the motivation for this PR? Add regression test cases of reclaiming buffer for both traditional and dynamic model #### How did you do it? 1. Adjust dynamic buffer test cases to verify reclaiming buffer in port admin down test 2. Add deployment test to verify correct profiles in `APPL_DB`, and `ASIC_DB` have been applied on each port. This is for both traditional and dynamic buffer models. 3. Adjust the script which transmits a switch from traditional model to dynamic model. 4. Shut down BGP neighbors before test and start up them during teardown as we need to run the test in topologies other than `ptf32`.
Description of PR
Summary: Test cases for reclaiming buffer on both traditional and dynamic model
It depends on all other PRs for reclaiming buffer
Signed-off-by: Stephen Sun stephens@nvidia.com
Type of change
Back port request
Approach
What is the motivation for this PR?
Add regression test cases of reclaiming buffer for both traditional and dynamic model
How did you do it?
APPL_DB
, andASIC_DB
have been applied on each port.This is for both traditional and dynamic buffer models.
ptf32
.How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation