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

internal [202205] Rename the platform_reboot to the pre_reboot_hook, remove the sysfs power cycle #40

Closed
wants to merge 2 commits into from

Conversation

vadymhlushko-mlnx
Copy link
Owner

@vadymhlushko-mlnx vadymhlushko-mlnx commented Nov 30, 2023

DEPENDS ON:

Why I did it

Add support for the graceful reboot instead of the sysfs power cycle to avoid filesystem corruption

Work item tracking
  • Microsoft ADO (number only):

How I did it

Rename the platform_reboot script to the pre_reboot_hook.
Remove the sysfs power cycle function, from now on the Debian reboot (/sbin/reboot) will be executed instead of the sysfs power cycle.

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

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

…emove sysfs power cycle, and Debina reboot

Signed-off-by: vadymhlushko-mlnx <[email protected]>
…r all mellanox devices

Signed-off-by: vadymhlushko-mlnx <[email protected]>
@vadymhlushko-mlnx vadymhlushko-mlnx changed the title internal 202205 graceful shutdown internal [202205] Rename the platform_reboot to the pre_reboot_hook, remove the sysfs power cycle Dec 13, 2023
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.

1 participant