[201911] Verify buffer priority groups for all platforms#4491
Merged
neethajohn merged 4 commits intosonic-net:201911from Nov 1, 2021
stephenxs:reclaim-buffer-201911-all-platform
Merged
[201911] Verify buffer priority groups for all platforms#4491neethajohn merged 4 commits intosonic-net:201911from stephenxs:reclaim-buffer-201911-all-platform
neethajohn merged 4 commits intosonic-net:201911from
stephenxs:reclaim-buffer-201911-all-platform
Conversation
Signed-off-by: Stephen Sun <[email protected]>
Signed-off-by: Stephen Sun <[email protected]>
|
This pull request introduces 1 alert when merging 047e2bc into 0639353 - view on LGTM.com new alerts:
|
Contributor
Author
|
/azpw run |
Signed-off-by: Stephen Sun <[email protected]>
|
This pull request fixes 1 alert when merging 2a9a33e into 0639353 - view on LGTM.com fixed alerts:
|
Signed-off-by: Stephen Sun <[email protected]>
|
This pull request fixes 1 alert when merging a610788 into 0639353 - view on LGTM.com fixed alerts:
|
Collaborator
|
@stephenxs are you able to make master branch test case also cover 201911 branch? |
Contributor
Yes. master branch can cover 201911 |
Collaborator
|
@yxieca and @neethajohn can you approve and sign off? |
neethajohn
approved these changes
Nov 1, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
Verify buffer priority groups for non-Mellanox platforms as well.
Signed-off-by: Stephen Sun [email protected]
How did you do it?
To verify:
How did you verify/test it?
It was verified on Mellanox platform. An old image without reclaiming buffer is adapted to simulate non-Mellanox platform.
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation