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] [arista] thermal info mismatch between platform.json and platform_api_conn #12629

Closed
wenyiz2021 opened this issue Nov 8, 2022 · 1 comment
Assignees
Labels
Arista Triaged this issue has been triaged

Comments

@wenyiz2021
Copy link
Contributor

wenyiz2021 commented Nov 8, 2022

Description

(Pdb) expected_num_thermals
5
(Pdb) chassis.get_all_thermals(platform_api_conn)
[{u'__module__': u'arista.utils.sonic_platform.thermal', u'__class__': u'Thermal', u'object_id': u'0x7f5d70e8cfd0'}, {u'__module__': u'arista.utils.sonic_platform.thermal', u'__class__': u'Thermal', u'object_id': u'0x7f5d70e8e070'}, {u'__module__': u'arista.utils.sonic_platform.thermal', u'__class__': u'Thermal', u'object_id': u'0x7f5d70e8e0d0'}, {u'__module__': u'arista.utils.sonic_platform.thermal', u'__class__': u'Thermal', u'object_id': u'0x7f5d70e8e130'}, {u'__module__': u'arista.utils.sonic_platform.thermal', u'__class__': u'Thermal', u'object_id': u'0x7f5d70e8e190'}, {u'__module__': u'arista.utils.sonic_platform.thermal', u'__class__': u'Thermal', u'object_id': u'0x7f5d70e8e1f0'}]
(Pdb) len(chassis.get_all_thermals(platform_api_conn))
6
(Pdb) duthost.facts.get("chassis").get('thermals')
[{u'name': u'Cpu temp sensor'}, {u'name': u'Center back'}, {u'name': u'Fap0 core0'}, {u'name': u'Fap0 core1'}, {u'name': u'PCIE'}]

FAILED platform_tests/api/test_chassis.py::TestChassisApi::test_thermals[str2-7804-lc6-1] - Failed: Number of thermals (6) does not match expected number (5)
FAILED platform_tests/api/test_chassis.py::TestChassisApi::test_thermals[str2-7804-lc7-1] - Failed: Number of thermals (6) does not match expected number (5)

Steps to reproduce the issue:

  1. run test_chassis.py::test_thermals on arista chassis

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 changed the title [chassis] [arista] thermal info mismatch [chassis] [arista] thermal info mismatch between platform.json and platform_api_conn Nov 8, 2022
@prgeor prgeor added Triaged this issue has been triaged Arista labels Nov 9, 2022
@wenyiz2021
Copy link
Contributor Author

PR #12714 merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arista Triaged this issue has been triaged
Projects
None yet
Development

No branches or pull requests

3 participants