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

[AS9716-32D]Support API2.0 to 202012BR #8411

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

Conversation

jostar-yang
Copy link
Contributor

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

Why I did it

Support API2.0 to 202012BR

How I did it

Implement needed code.

How to verify it

Test by show command.
root@sonic:~# show platform psustatus
PSU Model Serial Voltage (V) Current (A) Power (W) Status LED


PSU 1 YESM1300AM S0A000X601919000013 12.09 29.41 355.00 OK green
PSU 2 0.00 0.00 0.00 NOT OK red

root@sonic:/usr/local/bin# show platform fan
Drawer LED FAN Speed Direction Presence Status Timestamp


FanTray1 green FAN-1F 100% exhaust Present OK 20210321 18:03:44
FanTray1 green FAN-1R 100% exhaust Present OK 20210321 18:03:44
FanTray2 green FAN-2F 100% exhaust Present OK 20210321 18:03:44
FanTray2 green FAN-2R 100% exhaust Present OK 20210321 18:03:44
FanTray3 green FAN-3F 100% exhaust Present OK 20210321 18:03:44
FanTray3 green FAN-3R 100% exhaust Present OK 20210321 18:03:44
FanTray4 green FAN-4F 100% exhaust Present OK 20210321 18:03:44
FanTray4 green FAN-4R 100% exhaust Present OK 20210321 18:03:44
FanTray5 green FAN-5F 100% exhaust Present OK 20210321 18:03:44
FanTray5 green FAN-5R 100% exhaust Present OK 20210321 18:03:44
FanTray6 green FAN-6F 100% exhaust Present OK 20210321 18:03:44
FanTray6 green FAN-6R 100% exhaust Present OK 20210321 18:03:44
N/A green PSU-1 FAN-1 79% exhaust Present OK 20210321 18:03:44
N/A green PSU-2 FAN-1 0% exhaust Present OK 20210321 18:03

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 10, 2021

This pull request introduces 3 alerts when merging b9b32fa into 8b4f694 - view on LGTM.com

new alerts:

  • 3 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2021

This pull request introduces 2 alerts when merging a726e16 into 967d8ab - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Syntax error

@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

/easycla

1 similar comment
@jostar-yang
Copy link
Contributor Author

/easycla

@lgtm-com
Copy link

lgtm-com bot commented Sep 7, 2022

This pull request introduces 1 alert when merging 889d403 into edc4485 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request introduces 3 alerts when merging 25c2008 into a16843a - view on LGTM.com

new alerts:

  • 1 for Comparison using is when operands support `__eq__`
  • 1 for Except block handles 'BaseException'
  • 1 for Unused import

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