Skip to content

[Arista] [chassis] [platform] platform.json mismatch with platform_api_conn returned #12590

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

Closed
wenyiz2021 opened this issue Nov 2, 2022 · 1 comment
Assignees
Labels
Arista Chassis 🤖 Modular chassis support Triaged this issue has been triaged

Comments

@wenyiz2021
Copy link
Contributor

wenyiz2021 commented Nov 2, 2022

Description

• Test_chassis.py::test_modules
o str2-7804-lc6-1 WARNING pmon#platform_api_server.py: API 'get_module_index' not implemented
 warning though, but not seen on other LCs or SUP

• Test_chassis.py::test_get_name
 SUP has mismatch. Please update platform.json file, name should be DCS-7800-SUP1A, not DCS-7800A-SUP1A
 LC6 has name mismatch, but not platform.json issue, LC6 returns wrong name from platform_api_conn
• name = chassis.get_name(platform_api_conn)

• platform_tests/api/test_module.py::TestModuleApi::test_get_maximum_consumed_power
o Failed: Module 0 max consumed power format appears incorrect -> platform_api_conn: this needs to be a float number on all LCs and SUP

• platform_tests/api/test_psu.py::TestPsuApi::test_get_name[str2-7804-sup-1] FAILED
• platform_tests/api/test_psu.py::TestPsuApi::test_led[str2-7804-sup-1] FAILED
o These 2 failures are due to same reason that SUP has 12 PSUs, but:
 (Pdb) duthost.facts.get("chassis").get("psus")
 []
o If we want PSUs on SUP, then please update platform.json, provide psu names and LEDs for every PSUs on SUP, LCs are fine bc no PSUs on them,
o OR, self.num_psus = int(chassis.get_num_psus(platform_api_conn))should return 0 PSUs

• platform_tests/api/test_psu.py::TestPsuApi::test_power[str2-7804-sup-1]
o platform_api_conn:
o E Failed: PSU 2 reading does not make sense (power:0.0, voltage:0.0, current:0.0), PSU 2 is not operational, Voltage 0.0 of PSU 2 is not in between 11.699 and 12.898, PSU 3 reading does not make sense (power:0.0, voltage:0.0, current:0.0), PSU 3 is not operational, Voltage 0.0 of PSU 3 is not in between 11.699 and 12.898, PSU 4 reading does not make sense (power:0.0, voltage:0.0, current:0.0), PSU 4 is not operational, Voltage 0.0 of PSU 4 is not in between 11.699 and 12.898, PSU 5 reading does not make sense (power:0.0, voltage:0.0, current:0.0), PSU 5 is not operational, Voltage 0.0 of PSU 5 is not in between 11.699 and 12.898

• platform_tests/api/test_psu_fans.py::TestPsuFans::test_get_name[str2-7804-sup-1] FAILED
o missing psu fans name in platform.json

• platform_tests/api/test_psu_fans.py::TestPsuFans::test_get_speed[str2-7804-sup-1] FAILED
o platform_api_conn return 0 speed. Expecting psu fan speed 0-100

• platform_tests/api/test_psu_fans.py::TestPsuFans::test_set_fans_led[str2-7804-sup-1] FAILED
o psu_fan.get_status_led(platform_api_conn, j, i) always returns ‘off’ status

assigning to @Staphylo as this platform.json merged from #9717

Steps to reproduce the issue:

  1. run platform_tests/api/test_chassis.py::TestChassisApi::test_get_name

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):

@wenyiz2021 wenyiz2021 added Chassis 🤖 Modular chassis support Arista labels Nov 2, 2022
@wenyiz2021 wenyiz2021 changed the title [Arista] [chassis] platform.json names mismatch [Arista] [chassis] [platform] platform.json mismatch with platform_api_conn Nov 3, 2022
@wenyiz2021 wenyiz2021 changed the title [Arista] [chassis] [platform] platform.json mismatch with platform_api_conn [Arista] [chassis] [platform] platform.json mismatch with platform_api_conn returned Nov 3, 2022
@rlhui rlhui removed their assignment Nov 3, 2022
@prgeor prgeor added the Triaged this issue has been triaged label Nov 9, 2022
@wenyiz2021
Copy link
Contributor Author

#12714
#12667
#12627
fixed these info

@wenyiz2021 wenyiz2021 moved this to Done in SONiC Chassis Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arista Chassis 🤖 Modular chassis support Triaged this issue has been triaged
Projects
Archived in project
Development

No branches or pull requests

4 participants