fix some cel seastone platform api failed cases #237
Merged
Conversation
This file contains 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
1. failed cases in sfp module
2. failed cases in thermal module
|
@cel-eric pylint check yet? |
No introduce new pylint issue, I fixed some pylint warning in thermal, there still many remains in sfp module. |
root cause:
sensor file in /usr/share/sonic/platform is mount as ro,
do not write this file in pmon
mudsut4ke
approved these changes
Mar 3, 2021
mudsut4ke
merged commit 32af83c
into
SONIC-DEV:master_dx010_fix_failed_test_cases_of_api
mudsut4ke
pushed a commit
that referenced
this pull request
Jan 28, 2022
…t#9412) c2aac75 [SFP-Refactor] Fix LP mode API issue (#247) dba17c8 Firmware upgrade CLI support for QSFP-DD transceivers (#244) cd69212 [SFP-Refactor] Implement CMIS Low Power mode (#237) 9cea07f Fix RegGroupField decode (#245) 6ae1909 Add CMIS QSFP support (#246) c1f317d Gracefully handle CMIS APIs for passive modules (#238) ec7335d fix for firmware functions (#243) cf2ebe9 Fix RegBitField decode/encode (#242) ef4f2c6 Fix SFP_CABLE_TECH_FIELD (#240) e118644 remove time counting message in functions because function running time could be difficult to predict in unit tests (#241) Signed-off-by: Prince George <prgeor@microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
- What I did
- How I did it
- How to verify it
Run the Jenkins to show the new testbed platform test result.