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.

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

DellEMC: S6000, S6100, Z9332f - Update Fan LED, Thermal fields in platform.json #9459

Merged

Conversation

ArunSaravananBalachandran
Copy link
Contributor

Why I did it

To include newer Fan LED, thermal capabilities fields in platform.json of DellEMC S6000, S6100, Z9332f platforms.

How I did it

Add the capabilities fields in each platform's respective platform.json.

How to verify it

Ran sonic-mgmt platform api test cases that use capabilities fields and verified that the results are as expected.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

DellEMC: S6000, S6100, Z9332f - Update Fan LED, Thermal fields in platform.json

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

@prgeor prgeor requested a review from sujinmkang December 19, 2021 12:31
@sujinmkang sujinmkang merged commit a888a2d into sonic-net:master Jan 3, 2022
@qiluo-msft
Copy link
Collaborator

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

judyjoseph pushed a commit that referenced this pull request Jan 9, 2022
Why I did it
To include newer Fan LED, thermal capabilities fields in platform.json of DellEMC S6000, S6100, Z9332f platforms.

How I did it
Add the capabilities fields in each platform's respective platform.json.

How to verify it
Ran sonic-mgmt platform api test cases that use capabilities fields and verified that the results are as expected.
@ArunSaravananBalachandran ArunSaravananBalachandran deleted the platform_json branch January 13, 2022 07:45
@liuh-80
Copy link
Contributor

liuh-80 commented Feb 6, 2023

Change been merge to 202012 with following PR: #11772

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants