Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chassisd is pushing incorrect number of ASICs for linecards to CHASSIS_STAGE_DB #559

Open
vivekverma-arista opened this issue Nov 11, 2024 · 1 comment · May be fixed by #560
Open

chassisd is pushing incorrect number of ASICs for linecards to CHASSIS_STAGE_DB #559

vivekverma-arista opened this issue Nov 11, 2024 · 1 comment · May be fixed by #560

Comments

@vivekverma-arista
Copy link 

admin@nfc420:~$ sonic-db-cli CHASSIS_STATE_DB keys CHASSIS_MODULE_TABLE*
CHASSIS_MODULE_TABLE|LINE-CARD6
CHASSIS_MODULE_TABLE|LINE-CARD8
CHASSIS_MODULE_TABLE|LINE-CARD7
admin@nfc420:~$ sonic-db-cli CHASSIS_STATE_DB hgetall "CHASSIS_MODULE_TABLE|LINE-CARD6"
{'slot': '7', 'hostname': 'nfc420-7', 'num_asics': '0'}
admin@nfc420:~$ sonic-db-cli CHASSIS_STATE_DB hgetall "CHASSIS_MODULE_TABLE|LINE-CARD7"
{'slot': '8', 'hostname': 'nfc420-8', 'num_asics': '0'}
admin@nfc420:~$ sonic-db-cli CHASSIS_STATE_DB hgetall "CHASSIS_MODULE_TABLE|LINE-CARD8"
{'slot': '9', 'hostname': 'nfc420-9', 'num_asics': '0'}

Although these are linecards with one ASIC each.

There is a bug in this script where a variable defined inside a loop is being accessed outside of it. https://github.com/sonic-net/sonic-platform-daemons/blob/master/sonic-chassisd/scripts/chassisd#L361
@vivekverma-arista vivekverma-arista linked a pull request Nov 11, 2024 that will close this issue
Open
@arlakshm
Copy link
Contributor

Can you also create Test gap in the sonic-mgmt to catch this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
SONiC Chassis
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix chassisd bug vivekverma-arista/sonic-platform-daemons
2 participants
@arlakshm @vivekverma-arista