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

[sonic_platform/celestica]: Update Haliburton reboot cause API #5403

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

Conversation

mudsut4ke
Copy link
Contributor

@mudsut4ke mudsut4ke commented Sep 18, 2020

- Why I did it

  • Add more cases support in Haliburton reboot cause API.

- How I did it

  • Add Thermal Overload reboot cause support
  • Add new Watchdog reboot cause support

- How to verify it

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

  • 201811
  • 201911
  • 202006

@lgtm-com
Copy link

lgtm-com bot commented Sep 18, 2020

This pull request fixes 3 alerts when merging 2ab1338 into 2868a27 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Sep 18, 2020

This pull request fixes 3 alerts when merging 38541b1 into 2868a27 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Sep 18, 2020

This pull request fixes 3 alerts when merging ec1a892 into 2868a27 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Sep 22, 2020

This pull request fixes 3 alerts when merging 714fc5a into 03f82a0 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

@mudsut4ke mudsut4ke marked this pull request as ready for review September 22, 2020 08:50
@lgtm-com
Copy link

lgtm-com bot commented Oct 9, 2020

This pull request fixes 3 alerts when merging 60e5ccb into 42d9a44 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 9, 2020

This pull request fixes 3 alerts when merging 14a3cfe into c1add9a - view on LGTM.com

fixed alerts:

  • 3 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 14, 2020

This pull request fixes 3 alerts when merging 0fd7edf into 812e1a3 - view on LGTM.com

fixed alerts:

  • 3 for Unused import

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.

2 participants