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

Published 20 hours ago • verified publisher icon sonic-net

[chassisd] update chassisd to write fabric and lc asics on seperate table in chassis state db

Open arlakshm opened this issue 2 years ago • 2 comments

Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan [email protected]

Description

The following changes are done in chassisd

  • Change the ModuleUpdater init changed to include slot and supervisor as argument.
  • Add logic to in ModuleUpdater to update the asic_info in CHASSIS_FABRIC_ASIC_INFO for the supervisor and CHASSIS_ASIC_TABLE for the linecard asics respectively.
  • Add the linecard name is the key of the CHASSIS_ASIC_TABLE
  • update the UT

Motivation and Context

Fixes https://github.com/sonic-net/sonic-buildimage/issues/12575 and https://github.com/sonic-net/sonic-buildimage/issues/12575

How Has This Been Tested?

Update the pmon on the supervisor and linecard

Additional Information (Optional)

arlakshm avatar Nov 01 '22 21:11 arlakshm

@bmridul ++

prgeor avatar Nov 09 '22 19:11 prgeor

@arlakshm can you please paste the output of STATE_DB before and after this change in the PR description. Will be good for reference.

abdosi avatar Nov 11 '22 16:11 abdosi

@arlakshm can you please paste the output of STATE_DB before and after this change in the PR description. Will be good for reference.

@abdosi Updated the description with output after the change

arlakshm avatar Nov 15 '22 19:11 arlakshm

@abdosi , @prgeor can you please approve this pr if no other comments

arlakshm avatar Nov 15 '22 20:11 arlakshm

Hi @yxieca please merge this pr in 202205 https://github.com/sonic-net/sonic-buildimage/pull/12780 after cherry-picking this PR

arlakshm avatar Nov 27 '22 00:11 arlakshm