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

[AS4630-54PE] Add to support PDDF #8014

Merged
merged 8 commits into from
Feb 10, 2022

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang jostar_yang@accton.com.tw

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)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
@lgtm-com
Copy link

lgtm-com bot commented Jun 29, 2021

This pull request introduces 13 alerts when merging 3335a3b into 101b1fa - view on LGTM.com

new alerts:

  • 5 for Unused local variable
  • 2 for Testing equality to None
  • 2 for Unused import
  • 2 for Use of 'global' at module level
  • 1 for Unreachable code
  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Jun 30, 2021

This pull request introduces 3 alerts when merging 87aa7cd into 724d193 - view on LGTM.com

new alerts:

  • 2 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2021

This pull request introduces 2 alerts when merging 0865106 into 17f0217 - view on LGTM.com

new alerts:

  • 2 for Unused local variable

@geans-pin
Copy link
Contributor

/azpw run

1 similar comment
@jostar-yang
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jostar-yang
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@seanwu-ec seanwu-ec mentioned this pull request Dec 28, 2021
5 tasks
@prgeor prgeor merged commit f9a29ef into sonic-net:master Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants