-
Notifications
You must be signed in to change notification settings - Fork 650
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
[reboot]: Allow reboot to happen regardless vendor hook errors #3454
Conversation
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@prgeor Can you please help review this PR? Thanks |
@nazariig What is the connection between syncd service startup and your platform's reboot hook? |
@prgeor this bug is vendor specific. The flow can be interrupted if |
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@bingwang-ms can you take this to 202405 as a bug fix/improvement |
…-net#3454) * [reboot]: Allow reboot to happen regardless vendor hook errors. Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com> * [reboot]: Handle review comments. Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com> --------- Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
Cherry-pick PR to 202405: #3517 |
* [reboot]: Allow reboot to happen regardless vendor hook errors. Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com> * [reboot]: Handle review comments. Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com> --------- Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com
The fix is aimed to resolve reboot issue when vendor reboot hook is executed during
syncd
service startup.Relevant PRs:
What I did
How I did it
How to verify it
reboot
whensyncd
is starting and ASIC reset is in progressPrevious 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)