Skip to content

[Multi-ASIC] Added fast-reboot support for multi-asic - #4506

Open
abelamit wants to merge 7 commits into
sonic-net:masterfrom
abelamit:fast-reboot-multi-asic
Open

[Multi-ASIC] Added fast-reboot support for multi-asic#4506
abelamit wants to merge 7 commits into
sonic-net:masterfrom
abelamit:fast-reboot-multi-asic

Conversation

@abelamit

Copy link
Copy Markdown

What I did

Add fast-reboot support for multi-asic devices, according to this HLD - [Multi-ASIC] Added fast-reboot support

How I did it

Enhanced fast-reboot, fast-reboot-filter-routes.py, and generate_dump scripts to support fast-reboot multi-asic

How to verify it

by running fast-reboot on a multi-ASIC device and verifying that all ASICs perform fast-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)

Signed-off-by: Yair Raviv <yraviv@nvidia.com>
Signed-off-by: Yair Raviv <yraviv@nvidia.com>
Signed-off-by: Yair Raviv <yraviv@nvidia.com>
Signed-off-by: Yair Raviv <yraviv@nvidia.com>
Signed-off-by: Yair Raviv <yraviv@nvidia.com>
Signed-off-by: Yair Raviv <yraviv@nvidia.com>
Signed-off-by: Yair Raviv <yraviv@nvidia.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik

Copy link
Copy Markdown
Collaborator

@vaibhavhd kindly reminder to provide your feedback. if you wish to assing someone else on your behalf please do.

@rookie-who

Copy link
Copy Markdown
Contributor

Hi — this PR is flagged for cherry-pick to the 202605 branch, but it appears to be a new feature/enhancement rather than a bug fix. The 202605 branch is in stabilization, so we're limiting cherry-picks to bug fixes and critical platform support.

Could you please retarget this to the next release branch? If there's a specific customer requirement driving this for 202605, please comment with the justification and we can reconsider.

Comment by @vaibhavhd via automated tooling

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.

5 participants