-
Notifications
You must be signed in to change notification settings - Fork 850
[Mellanox] Ignore checking ASIC/module temperature if thermal updater is there #11165
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
[Mellanox] Ignore checking ASIC/module temperature if thermal updater is there #11165
Conversation
… is there Change-Id: I13ecc12308c546a23d845fb1f81d2f43b3d2fb15
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
… is there (sonic-net#11165) - Why I did it thermal updater is a thread which will update ASIC/module thermal sysfs nodes periodically. Currently, there is no way to stop thermal updater. So, there is no easy way to mock thermal sysfs value if thermal updater is there (thermal updater will always override mocked values). - How I did it Ignore ASIC/module thermal mock test if thermal updater is present - How to verify it Manual test sonic-mgmt regression test
Cherry-pick PR to 202311: #11858 |
… is there (#11165) - Why I did it thermal updater is a thread which will update ASIC/module thermal sysfs nodes periodically. Currently, there is no way to stop thermal updater. So, there is no easy way to mock thermal sysfs value if thermal updater is there (thermal updater will always override mocked values). - How I did it Ignore ASIC/module thermal mock test if thermal updater is present - How to verify it Manual test sonic-mgmt regression test
Why I did it
thermal updater is a thread which will update ASIC/module thermal sysfs nodes periodically. Currently, there is no way to stop thermal updater. So, there is no easy way to mock thermal sysfs value if thermal updater is there (thermal updater will always override mocked values).
Work item tracking
How I did it
Ignore ASIC/module thermal mock test if thermal updater is present
How to verify it
Manual test
sonic-mgmt regression test
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)