Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

execute a platform-provided pre-reboot script before shutting down swss #2708

Merged
merged 2 commits into from
Mar 8, 2023
Merged

Conversation

jhli-cisco
Copy link
Contributor

What I did

Give platform chance to do proper cleanup before shutting down swss in warm/fast-reboot by calling a platform-provided script

How I did it

update fast-reboot script.

How to verify it

execute fast-reboot/warm-reboot

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)

give platform chance to do proper cleanup before shutting down swss in warm/fast-reboot
@jhli-cisco jhli-cisco changed the title execute a platform-provided pre-shutdown script before shutting down swss execute a platform-provided pre-reboot script before shutting down swss Mar 1, 2023
@kevinskwang kevinskwang requested a review from vaibhavhd March 2, 2023 04:15
@jhli-cisco
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 2708 in repo sonic-net/sonic-utilities

@jhli-cisco
Copy link
Contributor Author

vaibhavhd, can you please review?

@rraghav-cisco
Copy link

@kevinwangsk , Pls merge, as @vaibhavhd has approved this.

@jhli-cisco
Copy link
Contributor Author

Hi Kevin, I am seeing "Only those with write access to this repository can merge pull requests." I dont see merge button.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants