-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[AS4630-54PE] Add to support PDDF #8014
Merged
prgeor
merged 8 commits into
sonic-net:master
from
jostar-yang:as4630-54pe_20210629_pddf
Feb 10, 2022
Merged
[AS4630-54PE] Add to support PDDF #8014
prgeor
merged 8 commits into
sonic-net:master
from
jostar-yang:as4630-54pe_20210629_pddf
Feb 10, 2022
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
Signed-off-by: Jostar Yang <[email protected]>
This pull request introduces 13 alerts when merging 3335a3b into 101b1fa - view on LGTM.com new alerts:
|
This pull request introduces 3 alerts when merging 87aa7cd into 724d193 - view on LGTM.com new alerts:
|
This pull request introduces 2 alerts when merging 0865106 into 17f0217 - view on LGTM.com new alerts:
|
/azpw run |
1 similar comment
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
5 tasks
prgeor
approved these changes
Feb 10, 2022
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.
Signed-off-by: Jostar Yang [email protected]
Why I did it
Support PDDF
How I did it
Implement PDDF related code
How to verify it
Test pddf cmd.
root@sonic:/usr/local/bin# pddf_psuutil seninfo
PSU Status Output Voltage (V) Output Current (A) Output Power (W) Temperature1 (C) Fan1 Speed (RPM)
PSU1 NOT OK 0.00 0.00 0.00 39.00 0
PSU2 OK 54.44 0.83 54.00 30.00 12896
root@sonic:/usr/local/bin#
root@sonic:/usr/local/bin#
root@sonic:/usr/local/bin# pddf_psuutil mfrinfo
PSU Status Manufacturer ID Model Serial Fan Airflow Direction
PSU1 NOT OK 3Y POWER YPEB1200AM SB020X181951000009 exhaust
PSU2 OK 3Y POWER YPEB1200AM SB020X181951000017 exhaust
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)