Skip to content

Revert "[Mellanox][Smartswitch] Set default reboot type as DPU reboot"#25079

Merged
liat-grozovik merged 3 commits intosonic-net:masterfrom
gpunathilell:rev_dpu_reb
Jan 29, 2026
Merged

Revert "[Mellanox][Smartswitch] Set default reboot type as DPU reboot"#25079
liat-grozovik merged 3 commits intosonic-net:masterfrom
gpunathilell:rev_dpu_reb

Conversation

@gpunathilell
Copy link
Contributor

This reverts commit ee76ce5.

Why I did it

As the original PR was created as a workaround for a driver issue, This commit is reverted, since now it is not requried anymore. This revert will change the behaviour during system reboot for DPUs dpus will start the startup process and then we proceed with switch reboot

Work item tracking
  • Microsoft ADO (number only):

How I did it

Revert commit

How to verify it

Execute reboot command

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

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)

@gpunathilell gpunathilell requested a review from lguohan as a code owner January 15, 2026 02:02
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

This reverts commit ee76ce5.

Signed-off-by: gpunathilell <gpunathilell@nvidia.com>
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: gpunathilell <gpunathilell@nvidia.com>
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit 4043f91 into sonic-net:master Jan 29, 2026
14 checks passed
xincunli-sonic pushed a commit to xincunli-sonic/sonic-buildimage that referenced this pull request Feb 2, 2026
sonic-net#25079)

This reverts commit ee76ce5.

- Why I did it
As the original PR was created as a workaround for a driver issue, This commit is reverted, since now it is not requried anymore. This revert will change the behaviour during system reboot for DPUs dpus will start the startup process and then we proceed with switch reboot

- How I did it
Revert commit

- How to verify it
Execute reboot command

Signed-off-by: gpunathilell <gpunathilell@nvidia.com>
Signed-off-by: Xincun Li <stli@microsoft.com>
FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Mar 6, 2026
sonic-net#25079)

This reverts commit ee76ce5.

- Why I did it
As the original PR was created as a workaround for a driver issue, This commit is reverted, since now it is not requried anymore. This revert will change the behaviour during system reboot for DPUs dpus will start the startup process and then we proceed with switch reboot

- How I did it
Revert commit

- How to verify it
Execute reboot command

Signed-off-by: gpunathilell <gpunathilell@nvidia.com>
Signed-off-by: Feng Pan <fenpan@microsoft.com>
dprital pushed a commit that referenced this pull request Mar 19, 2026
#25079)

This reverts commit ee76ce5.

- Why I did it
As the original PR was created as a workaround for a driver issue, This commit is reverted, since now it is not requried anymore. This revert will change the behaviour during system reboot for DPUs dpus will start the startup process and then we proceed with switch reboot

- How I did it
Revert commit

- How to verify it
Execute reboot command

Signed-off-by: gpunathilell <gpunathilell@nvidia.com>
Signed-off-by: dprital <drorp@nvidia.com>
@dgsudharsan
Copy link
Collaborator

@vmittal-msft Can you please help to cherry-pick this to 202511?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants