-
Notifications
You must be signed in to change notification settings - Fork 820
[Mellanox] Add minimal table definition for SN2201 #6943
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
Conversation
Signed-off-by: Kebo Liu <[email protected]>
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
561345c
to
bcf5f12
Compare
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
bcf5f12
to
448b718
Compare
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
pre-commit checker failure is not relevant to the new change. |
SN2201 have minimal table defined in PR sonic-net/sonic-buildimage#12602 test case also need to be updated. - How did you do it? Add minitable definition for SN2201 - How did you verify/test it? run test platform_tests/mellanox/test_thermal_control.py::test_dynamic_minimum_table - Any platform specific information? Mellanox only Signed-off-by: Kebo Liu <[email protected]>
Signed-off-by: Kebo Liu [email protected]
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
SN2201 have minimal table defined in PR sonic-net/sonic-buildimage#12602
test case also need to be updated.
How did you do it?
Add minitable definition for SN2201
How did you verify/test it?
run test platform_tests/mellanox/test_thermal_control.py::test_dynamic_minimum_table
Any platform specific information?
Mellanox only
Supported testbed topology if it's a new test case?
Documentation