Skip to content

Fix for #23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes#645

Merged
judyjoseph merged 5 commits intosonic-net:masterfrom
rameshraghupathy:master-23205
Aug 5, 2025
Merged

Fix for #23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes#645
judyjoseph merged 5 commits intosonic-net:masterfrom
rameshraghupathy:master-23205

Conversation

@rameshraghupathy
Copy link
Contributor

@rameshraghupathy rameshraghupathy commented Jul 17, 2025

Fix for #23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes

Description

Fixes #23205
The corresponding sonic-utiltiies commit is: sonic-net/sonic-utilities#3984

Motivation and Context

Some recent design changes not initializing some fields in DPU_STATE and DPU_MODULE_INFO tables.

How Has This Been Tested?

initializing the fields in DPU_STATE and DPU_MODULE_INFO tables.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@gpunathilell
Copy link
Contributor

Fixes: sonic-net/sonic-buildimage#23205

@gpunathilell
Copy link
Contributor

@rameshraghupathy the case mentioned in the issue (Removal of pre-shutdown call during initialization:
The Dpu initialization pre_shutdown call was removed as part of the sonic-platform-daemons PR introduction) is not handled as part of this PR. Please fix that as well

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rameshraghupathy
Copy link
Contributor Author

@rameshraghupathy the case mentioned in the issue (Removal of pre-shutdown call during initialization: The Dpu initialization pre_shutdown call was removed as part of the sonic-platform-daemons PR introduction) is not handled as part of this PR. Please fix that as well

@gpunathilell Can you check now?

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rameshraghupathy
Copy link
Contributor Author

@vvolam @gpunathilell @oleksandrivantsiv Please take a look

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld added a commit to mssonicbld/sonic-utilities that referenced this pull request Aug 5, 2025
… chassisd/sonic-utiltiies changes for consecutive admin state changes

Fix for #23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes

Description
Fixes #23205
The corresponding sonic-platform-daemon commit is: # sonic-net/sonic-platform-daemons#645

Motivation and Context
Some recent design changes not initializing some fields in DPU_STATE and DPU_MODULE_INFO tables.

How Has This Been Tested?
initializing the fields in DPU_STATE and DPU_MODULE_INFO tables
@judyjoseph judyjoseph merged commit 7b347c6 into sonic-net:master Aug 5, 2025
5 checks passed
mssonicbld added a commit to sonic-net/sonic-utilities that referenced this pull request Aug 5, 2025
… chassisd/sonic-utiltiies changes for consecutive admin state changes (#4015)

Fix for #23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes

Description
Fixes #23205
The corresponding sonic-platform-daemon commit is: # sonic-net/sonic-platform-daemons#645

Motivation and Context
Some recent design changes not initializing some fields in DPU_STATE and DPU_MODULE_INFO tables.

How Has This Been Tested?
initializing the fields in DPU_STATE and DPU_MODULE_INFO tables
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #656

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.

10 participants