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

[Mellanox] Enhancement for support PSU LED management #12

Closed
wants to merge 2 commits into from
Closed

[Mellanox] Enhancement for support PSU LED management #12

wants to merge 2 commits into from

Conversation

Junchao-Mellanox
Copy link
Owner

- Why I did it

PSU LED management require PSU temperature, PSU temperature threshold, PSU voltage min value and PSU voltage max value to determine PSU LED status.

- How I did it

Implement following API in PSU class for upper layer get information for LED management:

  1. get_temperature
  2. get_temperature_high_threshold
  3. get_voltage_high_threshold (not implemented, hw-mgmt doesn't provide related sysfs for now)
  4. get_voltage_low_threshold (not implemented, hw-mgmt doesn't provide related sysfs for now)

- How to verify it

  1. Manual test on Spectrum1, Spectrum2
  2. Add/adjust test cases in sonic-mgmt and run the case

- Description for the changelog

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

@Junchao-Mellanox
Copy link
Owner Author

sonic-net/sonic-platform-common#84

@Junchao-Mellanox Junchao-Mellanox deleted the psu-led branch May 7, 2020 10:04
Junchao-Mellanox pushed a commit that referenced this pull request Jul 31, 2020
@jleveque
[sonic-telemetry][sonic-mgmt-common] Update submodules (sonic-net#4986)
2890670 
* src/sonic-telemetry fa8d498...3bd7ca3 (4):
  > Update gnmi deps (#40)
  > [testdata] Update SFP keys to align with new standard (#39)
  > Fixed the parameters for subscribe APIs (#38)
  > Azure ro mode (#34)

* src/sonic-mgmt-common 444aa9a...cc01ce4 (4):
  > Make gnmi dep version the same as in telemetry repo (#17)
  > Cleanup translib and cvl go test cases (#13)
  > Package update and enhancements/fixes in YGOT, and Request Binder (#12)
  > Translib phase I changes (#11)

Note: sonic-telemetry submodule update is dependent upon sonic-mgmt-common submodule update, thus updating both in this patch
Junchao-Mellanox pushed a commit that referenced this pull request Nov 12, 2021
@alexrallen @saiarcot895
Update pointer (#12)
cc5a2f3 
Updated the hw-mgmt pointer to include some bugfixes related to power supply voltages.
Junchao-Mellanox pushed a commit that referenced this pull request Mar 13, 2024
@mssonicbld
[submodule] Update submodule sonic-dash-api to the latest HEAD automa…
fd08edf 
…tically (sonic-net#17847)

#### Why I did it
src/sonic-dash-api
```
* 8f481de - (HEAD -> master, origin/master, origin/HEAD) [misc]: Add utils CLI (#12) (24 hours ago) [Ze Gan]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@keboliu keboliu keboliu approved these changes

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
@Junchao-Mellanox @keboliu