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

[AS5835-54X] Fix chassis health led bug #8276

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

Conversation

jostar-yang
Copy link
Contributor

@jostar-yang jostar-yang commented Jul 28, 2021

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

Why I did it

Chassis health led has bug and it will let show led fail when health found PSU is not power.

How I did it

Fix chassis.py code.

How to verify it

Plug-out one PSU and check "show system-health summary". Led should how amber

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)

Jostar Yang added 2 commits July 28, 2021 17:19
@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).

@lgtm-com
Copy link

lgtm-com bot commented Oct 15, 2021

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

new alerts:

  • 2 for Syntax error

@lgtm-com
Copy link

lgtm-com bot commented Jan 20, 2022

This pull request introduces 1 alert when merging 791178f into 2eb8fe3 - 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

@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.

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request introduces 2 alerts when merging f19d8a7 into aa4379d - view on LGTM.com

new alerts:

  • 2 for Unused import

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 5, 2022

CLA Missing ID CLA Not Signed

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