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

[DellEMC] S6100 Reboot cause determining CPU reset in fast-reboot #8024

Merged
merged 2 commits into from
Aug 4, 2021

Conversation

santhosh-kt
Copy link
Contributor

@santhosh-kt santhosh-kt commented Jun 30, 2021

Why I did it

  • Added a check in determining CPU reset in fast-/warm-reboot in their respective platform plugin.
  • Introducing reboot plugin for "reboot" command to handle its own platform plugin.

How I did it

On branch s6100_fast_warm_check
Changes to be committed:
(use "git reset HEAD ..." to unstage)

    modified:   ../../debian/platform-modules-s6100.install
    modified:   ../scripts/fast-reboot_plugin
    modified:   ../scripts/platform_reboot_override
    new file:   ../scripts/reboot_plugin
    modified:   ../scripts/track_reboot_reason.sh
    modified:   chassis.py

How to verify it

  • Triggered cold-reset inside fast-reboot to test out the reboot-cause 2.0 API.

s6100-logs.txt
OS9 to SONIC - cold reset.txt
OS9 to SONIC - fastboot.txt

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)

@santhosh-kt santhosh-kt marked this pull request as ready for review July 6, 2021 12:44
@santhosh-kt santhosh-kt requested a review from lguohan as a code owner July 6, 2021 12:44
@lguohan lguohan requested a review from sujinmkang July 6, 2021 22:15
@lguohan
Copy link
Collaborator

lguohan commented Jul 6, 2021

@sujinmkang , can you review?

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sujinmkang sujinmkang merged commit e5a6446 into sonic-net:master Aug 4, 2021
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
…nic-net#8024)

Why I did it
Added a check in determining CPU reset in fast-/warm-reboot in their respective platform plugin.
Introducing reboot plugin for "reboot" command to handle its own platform plugin.
How I did it
On branch s6100_fast_warm_check
Changes to be committed:
(use "git reset HEAD ..." to unstage)

    modified:   ../../debian/platform-modules-s6100.install
    modified:   ../scripts/fast-reboot_plugin
    modified:   ../scripts/platform_reboot_override
    new file:   ../scripts/reboot_plugin
    modified:   ../scripts/track_reboot_reason.sh
    modified:   chassis.py
How to verify it
Triggered cold-reset inside fast-reboot to test out the reboot-cause 2.0 API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants