Skip to content

[chassis] modify chassisd to add fabric card information to PHYSICAL_ENTITY_INFO table only if module is present #646

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vishal-nokia
Copy link

Description

Update chassisd to add fabric card module information to PHYSICAL_ENTITY_INFO table in STATE_DB only if the module is present.

Motivation and Context

Fixes #23372

How Has This Been Tested?

Run sonic-db-cli STATE_DB keys "PHYSICAL_ENTITY_INFO|FABRIC*"`
It should only list the fabric card modules that are present.

Additional Information (Optional)

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vishal-nokia vishal-nokia force-pushed the fix_update_entity_info branch from 588cc32 to c98104a Compare July 18, 2025 22:51
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi
Copy link
Contributor

abdosi commented Jul 23, 2025

@judyjoseph @ymd-arista : please check.

…present

- modify module_db_update function to add module to PHYSICAL_ENTITY_INFO table
  in STATE_DB only if module is present
- Remove entry from PHYSICAL_ENTITY_INFO table if card is not present
- add test case to check fabric removal scenario
@vishal-nokia vishal-nokia force-pushed the fix_update_entity_info branch from c98104a to 8b8cd97 Compare July 24, 2025 13:44
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Bug: snmp/test_snmp_phy_entity.py::test_fabric_card_info fails due to mismatch in SNMP info and STATE_DB
4 participants