Skip to content

[202311][graceful reboot] Add the pre_reboot_hook script execution, add the watchdog arm before the reboot#3204

Merged
yxieca merged 1 commit intosonic-net:202311from
vadymhlushko-mlnx:202311-graceful-shutdown
Apr 12, 2024
Merged

[202311][graceful reboot] Add the pre_reboot_hook script execution, add the watchdog arm before the reboot#3204
yxieca merged 1 commit intosonic-net:202311from
vadymhlushko-mlnx:202311-graceful-shutdown

Conversation

@vadymhlushko-mlnx
Copy link
Copy Markdown
Contributor

@vadymhlushko-mlnx vadymhlushko-mlnx commented Mar 11, 2024

DEPENDS ON: [202311][graceful reboot] Rename the platform_reboot to the pre_reboot_hook, remove the sysfs power cycle #18325

What I did

Add the pre_reboot_hook script execution, add the watchdog arm before the reboot, in order to support the graceful reboot instead of the sysfs power cycle, to avoid filesystem corruption.

How I did it

Modify the scripts/reboot

How to verify it

  1. Start watching logs by using show log -f and journalctl -p debug -f
  2. Execute the reboot command from the switch CLI
  3. Check in logs that all systemd services terminated correctly

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

… --force flag bug

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>
@liat-grozovik
Copy link
Copy Markdown
Collaborator

@yxieca kindly reminder, this is a major fix needed for 202311

@vadymhlushko-mlnx
Copy link
Copy Markdown
Contributor Author

vadymhlushko-mlnx commented Apr 11, 2024

@yxieca, @prgeor kind reminder to merge, the same changes were already merged into master

@yxieca yxieca merged commit 21f69fb into sonic-net:202311 Apr 12, 2024
liat-grozovik pushed a commit that referenced this pull request Sep 3, 2024
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR #3203
The same PR for 202311 doesn't have this mistake - #3204

- How I did it

- How to verify it
Call reboot -f
mssonicbld pushed a commit to mssonicbld/sonic-utilities that referenced this pull request Sep 6, 2024
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR sonic-net#3203
The same PR for 202311 doesn't have this mistake - sonic-net#3204

- How I did it

- How to verify it
Call reboot -f
mssonicbld pushed a commit that referenced this pull request Sep 6, 2024
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR #3203
The same PR for 202311 doesn't have this mistake - #3204

- How I did it

- How to verify it
Call reboot -f
jianyuewu pushed a commit to jianyuewu/sonic-utilities that referenced this pull request Feb 7, 2025
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR sonic-net#3203
The same PR for 202311 doesn't have this mistake - sonic-net#3204

- How I did it

- How to verify it
Call reboot -f
jianyuewu pushed a commit to jianyuewu/sonic-utilities that referenced this pull request Feb 7, 2025
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR sonic-net#3203
The same PR for 202311 doesn't have this mistake - sonic-net#3204

- How I did it

- How to verify it
Call reboot -f
jianyuewu pushed a commit to jianyuewu/sonic-utilities that referenced this pull request Feb 8, 2025
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR sonic-net#3203
The same PR for 202311 doesn't have this mistake - sonic-net#3204

- How I did it

- How to verify it
Call reboot -f

Signed-off-by: Jianyue Wu <jianyuew@nvidia.com>
jianyuewu pushed a commit to jianyuewu/sonic-utilities that referenced this pull request Feb 10, 2025
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR sonic-net#3203
The same PR for 202311 doesn't have this mistake - sonic-net#3204

- How I did it

- How to verify it
Call reboot -f
nmoray pushed a commit to nmoray/sonic-utilities that referenced this pull request Jun 25, 2025
- What I did
Add back the support to call reboot -f
The support for this option was accidentally removed as part of this PR sonic-net#3203
The same PR for 202311 doesn't have this mistake - sonic-net#3204

- How I did it

- How to verify it
Call reboot -f
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.

3 participants