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.

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

[chassis] test_show_patform_summary test fails for Sup #17819

Closed
anamehra opened this issue Jan 18, 2024 · 3 comments
Closed

[chassis] test_show_patform_summary test fails for Sup #17819

anamehra opened this issue Jan 18, 2024 · 3 comments
Assignees
Labels
Chassis 🤖 Modular chassis support

Comments

@anamehra
Copy link
Contributor

Description

I see get_platform_summary was modified by this PR #16539 and now the AISC count is returned based on asic presence.
Earlier on Supervisor, 'show platform summary' was used to display the max asic supported, which is 16 for the cisco-8000 chassis supervisor. Now it returns the number based on the number FCs inserted.

In sonic-mgmt lab file, we are using num_asics: 16, for max asics supported, and the presence list is defined by the asic_present field.

This is causing the test case to fail.

>                     "Unexpected value of fields, actual={}, expected={} on host '{}'".format(actual_fields_values, expected_fields_values, duthost.hostname))
E       Failed: Unexpected value of fields, actual=set([u'x86_64-8800_rp_o-r0', u'FOC2149NUED', u'4', u'8800-RP-O', u'0.12', u'Cisco-8800-RP', u'chassis-packet', u'cisco-8000']), expected=set([u'Cisco-8800-RP', '16', u'cisco-8000', None]) on host 'sfd-01'

actual_fields = set(['ASIC', 'ASIC Count', 'Hardware Revision', 'HwSKU', 'Model Number', 'Platform', ...])
actual_fields_values = set(['0.12', '4', '8800-RP-O', 'Cisco-8800-RP', 'FOC2149NUED', 'chassis-packet', ...])
cmd        = 'show platform summary'
diff_fields_values = set([None, '16'])

Changing the num_asics in lab file to match the number of asic presence does not work and breaks minigraph deployment.

Steps to reproduce the issue:

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@anamehra
Copy link
Contributor Author

@abdosi , @vperumal , for your viz

@abdosi
Copy link
Contributor

abdosi commented Jan 29, 2024

@anamehra working on fix on sonic-mgmt side.

@anamehra
Copy link
Contributor Author

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chassis 🤖 Modular chassis support
Projects
Archived in project
Development

No branches or pull requests

3 participants