Skip to content

[Warm Restart] Add waitWarmBootStarted to restartWaiter#1161

Closed
zili11720 wants to merge 3 commits into
sonic-net:masterfrom
zili11720:wait_warm_reboot_started
Closed

[Warm Restart] Add waitWarmBootStarted to restartWaiter#1161
zili11720 wants to merge 3 commits into
sonic-net:masterfrom
zili11720:wait_warm_reboot_started

Conversation

@zili11720
Copy link
Copy Markdown

@zili11720 zili11720 commented Mar 23, 2026

Add WaitWarmRebootStarted to restartWaiter to provide an event-driven indicator for when a warm reboot begins.

This PR is related to:
sonic-net/sonic-sairedis#1810
sonic-net/sonic-buildimage#26332

Together, these PRs introduce an event-driven mechanism that starts and stops phcsync.sh during a warm reboot.

Order of merge:

  1. [Warm Restart] Add waitWarmBootStarted to restartWaiter #1161
  2. [Mellanox] Add phcsync warm reboot gate to avoid sync during warm reboot sonic-buildimage#26332
  3. [Mellanox] Activate phcsync gate to prevent clock sync during warm reboot sonic-sairedis#1810

Signed-off-by: Zili Bombach <zbombach@nvidia.com>
Signed-off-by: Zili Bombach <zbombach@nvidia.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Zili Bombach <zbombach@nvidia.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@zili11720
Copy link
Copy Markdown
Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik marked this pull request as draft April 21, 2026 11:50
@zili11720 zili11720 changed the title [Mellanox] Add waitWarmBootStarted to restartWaiter Add waitWarmBootStarted to restartWaiter May 6, 2026
@zili11720 zili11720 changed the title Add waitWarmBootStarted to restartWaiter [Warm Restart] Add waitWarmBootStarted to restartWaiter May 6, 2026
@zili11720 zili11720 closed this May 20, 2026
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