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

[AS7712_32X] Support API2.0 to 202012BR #8617

Open
wants to merge 17 commits into
base: 202012
Choose a base branch
from

Conversation

jostar-yang
Copy link
Contributor

@jostar-yang jostar-yang commented Aug 27, 2021

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

Why I did it

Support API2.0 to 202012 Branch

How I did it

Implement needed code.

How to verify it

Test via show cmd.
show platform psustatus
PSU Model Serial Voltage (V) Current (A) Power (W) Status LED


PSU 1 N/A N/A N/A NOT PRESENT red
PSU 2 YM-2651Y SA160N091539046565 11.921 10.75 127.0 OK green

root@as7712-32x:/home/admin# show platform temperature
Sensor Temperature High TH Low TH Crit High TH Crit Low TH Warning Timestamp


PSU-1 temp sensor 1 0 80 N/A N/A N/A False 20210320 00:55:16
PSU-2 temp sensor 1 28 80 N/A N/A N/A False 20210320 00:55:16
Temp sensor 1 28.5 80 N/A N/A N/A False 20210320 00:55:16
Temp sensor 2 26 80 N/A N/A N/A False 20210320 00:55:16
Temp sensor 3 29 80 N/A N/A N/A False 20210320 00:55:16
Temp sensor 4 25.5 80 N/A N/A N/A False 20210320 00:55:16

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

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 Aug 27, 2021

This pull request introduces 1 alert when merging 25c5735 into 06b9d3a - view on LGTM.com

new alerts:

  • 1 for Unused import

@jostar-yang
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@jostar-yang
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 4, 2022

CLA Missing ID CLA Not Signed

@jostar-yang
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@jostar-yang
Copy link
Contributor Author

/easycla

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request introduces 1 alert when merging db140cc into a16843a - view on LGTM.com

new alerts:

  • 1 for Except block handles 'BaseException'

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.

2 participants