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

[Edgecore][AS5812-54x]Support API2.0 #11905

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: jostar-yang jostar_yang@edge-core.com

Why I did it

Support new platform api for API2.0

How I did it

Implement needed code.

How to verify it

Test show platform cmd.
root@sonic:/home/admin# show platform psustatus
PSU Model Serial HW Rev Voltage (V) Current (A) Power (W) Status LED

PSU 1 CPR-4011-4M11 N/A 0.00 0.00 0.00 NOT OK red
PSU 2 CPR-4011-4M11 4011411H35T0411 N/A 12.00 6.25 75.00 OK green
root@sonic:/home/admin#
root@sonic:/home/admin# show platform psustatus
PSU Model Serial HW Rev Voltage (V) Current (A) Power (W) Status LED

PSU 1 CPR-4011-4M11 N/A 0.00 0.00 0.00 NOT OK red
PSU 2 CPR-4011-4M11 4011411H35T0411 N/A 12.00 6.19 74.00 OK green
root@sonic:/home/admin#
root@sonic:/home/admin# show platform temp
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 20220811 09:44:28
PSU-2 temp sensor 1 30 80 N/A N/A N/A False 20220811 09:44:28
Temp sensor 1 25.5 80 N/A N/A N/A False 20220811 09:44:28
Temp sensor 2 23.5 80 N/A N/A N/A False 20220811 09:44:28
Temp sensor 3 22 80 N/A N/A N/A False 20220811 09:44:28

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
    --> Customer need this patch.
  • 202205
    --> Customer need this patch.

Description for the changelog

Link to config_db schema for YANG module changes

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

Signed-off-by: jostar-yang <jostar_yang@edge-core.com>
@yejianquan
Copy link
Contributor

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request introduces 1 alert when merging 000d19f into 95f4af3 - view on LGTM.com

new alerts:

  • 1 for Except block handles 'BaseException'

@jostar-yang jostar-yang changed the title [Edge-core][AS5812-54x]Support API2.0 [Edgecore][AS5812-54x]Support API2.0 Nov 24, 2022
@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.

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

Successfully merging this pull request may close these issues.

6 participants