sonic-platform-daemons icon indicating copy to clipboard operation
sonic-platform-daemons copied to clipboard

Added SmartSwitch support in chassisd and enabling chassisd

Open rameshraghupathy opened this issue 10 months ago • 9 comments

Added SmartSwitch support in chassisd and enabling chassisd for fixed SmartSwitches

Description

chassisd is enabled only for modular chassis. Smartswitch is a fixed chassis. However it has been treated like a modular chassis to manage the DPU cards just like the line-cards of a modular chassis. chassisd will be enabled only on the smartswitch NPU and hence the scope of these changes are limited only to NPU and not applicable to DPU.

Motivation and Context

chassisd is enabled only for modular chassis. Smartswitch is a fixed chassis. However it has been treated like a modular chassis to manage the DPU cards just like the line-cards of a modular chassis. Hence, chassid is needed for SmartSwitch and enabled here. Also, some of the table updates and clean up are not required for SmartSwitch platform and hence using the is_smartswitch API to selectively enable it. the text "fixes #xxxx", "closes #xxxx" or "resolves #xxxx" here

How Has This Been Tested?

  1. Enabled and built and image and checked if the chassisd is running all the time without crashing
  2. Verify if the config change handler is working as expected by issuing config CLIs to startup and shutdown DPUs

Additional Information (Optional)

rameshraghupathy avatar Apr 15 '24 03:04 rameshraghupathy

What are the states supported by the DPUs in the Smart Switch?

oleksandrivantsiv avatar Jun 11 '24 01:06 oleksandrivantsiv

What are the states supported by the DPUs in the Smart Switch?

”dpu_midplane_link_state” ”dpu_control_plane_state" "dpu_data_plane_state"

rameshraghupathy avatar Jun 11 '24 02:06 rameshraghupathy

@rameshraghupathy can you point to the Yang model change for admin_status for DPU in config DB table?

prgeor avatar Sep 10 '24 18:09 prgeor

@rameshraghupathy Please specify in the PR description if this chassisd changes is limited to NPU or DPU or both

prgeor avatar Sep 10 '24 19:09 prgeor