Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[sonic_platform]: Fixed the failed test case of Seastone-DX010's Fan API. #238

Merged
merged 2 commits into from
Mar 5, 2021
Merged

[sonic_platform]: Fixed the failed test case of Seastone-DX010's Fan API. #238

merged 2 commits into from
Mar 5, 2021

Conversation

mudsut4ke
Copy link

@mudsut4ke mudsut4ke commented Mar 3, 2021 •
edited
Loading

- What I did

  • Fixed the failed test case of Seastone-DX010's Fan API

- How I did it

  • Update Fan APIs follow latest specification
  • Update fan configuration in platform.json
  • Add fan_drawer APIs

- How to verify it

  • Verify platform_tests result from SONiC testbed

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@mudsut4ke mudsut4ke marked this pull request as ready for review March 5, 2021 10:23
@mudsut4ke mudsut4ke merged commit 85362d8 into SONIC-DEV:master_dx010_fix_failed_test_cases_of_api Mar 5, 2021
@mudsut4ke mudsut4ke deleted the master_dx010_fix_failed_test_cases_of_api_fan branch March 5, 2021 10:23
@nasbdh9
Copy link

nasbdh9 commented Mar 12, 2021

When testing these submitted build, the fan indicator is still red. Is this normal?

mudsut4ke pushed a commit that referenced this pull request Jan 28, 2022
@prgeor
[submodule-update] Updating sonic-platform-common submodule (sonic-ne…
b051a9f 
…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>
pphuchar pushed a commit that referenced this pull request Apr 18, 2022
@vdahiya12
[202012][sonic-platform-daemons] submodule update (sonic-net#10121)
8ad3678 
4dc2d0a (HEAD -> 202012, origin/202012) Clear all data from DB table when the daemon stops (#228)
1df28d7 [ycabled] increase UT coverage of ycabled daemon (#238)

Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mudsut4ke @nasbdh9