matthew/sonic-buildimage
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[202012] [Arista] Raise ValueError on thermal manager invalid fan speed (#8903)

Browse Source 
Why I did it
Vaibhav Dahiya notified me that invalid fan speed policy was expecting an error raised in sonic-mgmt testing, but it was not raised.
This change will fix test_platform_info.py::test_thermal_control_load_invalid_value_json

How I did it
Add in the suggested code chunk to Arista platform submodule to raise ValueError when an invalid fan speed is set in thermal policy.

How to verify it
Vaibhav Dahiya has verified it through sonic-mgmt testing.

Co-authored-by: Zhi Yuan (Carl) Zhao <zyzhao@arista.com>
This commit is contained in:
zzhiyuan 2021-10-05 18:29:26 -07:00 committed by GitHub
parent d69564a1e7
commit 380ad2e8fc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
platform/barefoot/sonic-platform-modules-arista

@ -1 +1 @@
Subproject commit 7304dbebdd80d7278abc6136e1df73190d405483
Subproject commit cf36e513e41cd10a6d6aad5bcf71ef7871a2806d

2
platform/broadcom/sonic-platform-modules-arista

@ -1 +1 @@
Subproject commit 7304dbebdd80d7278abc6136e1df73190d405483
Subproject commit cf36e513e41cd10a6d6aad5bcf71ef7871a2806d