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

[fast-reboot] Update reboot reason file to indicate fast-reboot was called #277

Merged
merged 2 commits into from
Jun 30, 2018
Merged

[fast-reboot] Update reboot reason file to indicate fast-reboot was called #277

merged 2 commits into from
Jun 30, 2018

Conversation

jleveque
Copy link
Contributor

No description provided.

@jleveque jleveque self-assigned this Jun 30, 2018
# Wait until all buffers synced with disk
sync
sleep 1
sync

# Reboot: explicity call Linux native reboot under sbin
echo "Rebooting to $NEXT_SONIC_IMAGE..."
/sbin/reboot
exec /sbin/reboot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure for this exec. Did you test it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. I made the same change recently in the scripts/reboot script and tested that thoroughly.

@jleveque jleveque merged commit 9c4c362 into sonic-net:master Jun 30, 2018
@jleveque jleveque deleted the fast_reboot_reason branch June 30, 2018 01:18
madhupalu added a commit to madhupalu/sonic-utilities that referenced this pull request Jan 25, 2019
…el or portchannel members only when it is configured (sonic-net#277)
prsunny pushed a commit that referenced this pull request Jan 28, 2019
…el or portchannel members only when it is configured (#277) (#445)
madhupalu added a commit to madhupalu/sonic-utilities that referenced this pull request Jan 30, 2019
…et#277)

Signed-off-by: madhu Pal <madhupa@aviznetworks.com>
@madhupalu madhupalu mentioned this pull request Jan 30, 2019
prsunny added a commit that referenced this pull request Jan 30, 2019
…ortchannel or portchannel members only when it is configured (#277) (#445)"

This reverts commit dc5d5c4.
prsunny added a commit that referenced this pull request Jan 30, 2019
…ortchannel or portchannel members only when it is configured (#277) (#445)" (#452)

This reverts commit dc5d5c4.
madhupalu added a commit to madhupalu/sonic-utilities that referenced this pull request Feb 3, 2019
…members add/delete

                     only when it is configured (sonic-net#277)

Signed-off-by: madhu Pal <madhupa@aviznetworks.com>
madhupalu added a commit to madhupalu/sonic-utilities that referenced this pull request Feb 12, 2019
…red (sonic-net#277)

Signed-off-by: madhu Pal <madhupa@aviznetworks.com>
madhupalu added a commit to madhupalu/sonic-utilities that referenced this pull request Feb 12, 2019
…erations, covered vendor interfaces as well. (sonic-net#277)

Signed-off-by: madhu Pal <madhupa@aviznetworks.com>
madhupalu added a commit to madhupalu/sonic-utilities that referenced this pull request Feb 17, 2019
…tempting delete operations (sonic-net#277)

Signed-off-by: madhu Pal <madhupa@aviznetworks.com>
mihirpat1 pushed a commit to mihirpat1/sonic-utilities that referenced this pull request Sep 15, 2023
- Description
Add some new reboot causes to cover followinging scenarios:
BIOS - In case the BIOS upgrade process ended with failure and cause the switch to reset.
CPU - Reset is initiated by SW on the CPU. it could be that SW encountered some catastrophic situation like a memory leak, eventually, the kernel reset the whole switch.
Push button - Reset by pushing the reset button
Reset from ASIC - Reset which is caused by ASIC.
Motivation and Context
Add more reboot causes to cover more scenarios.

- How Has This Been Tested?
UT is added with the code change.
Run community reboot test to see the reboot cause checker can pass.

Signed-off-by: Kebo Liu <kebol@nvidia.com>
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.

None yet

3 participants