From 380ad2e8fc19f5d619ba7658c19be225acfbff94 Mon Sep 17 00:00:00 2001 From: zzhiyuan Date: Tue, 5 Oct 2021 18:29:26 -0700 Subject: [PATCH] [202012] [Arista] Raise ValueError on thermal manager invalid fan speed (#8903) 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 --- platform/barefoot/sonic-platform-modules-arista | 2 +- platform/broadcom/sonic-platform-modules-arista | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/platform/barefoot/sonic-platform-modules-arista b/platform/barefoot/sonic-platform-modules-arista index 7304dbebdd..cf36e513e4 160000 --- a/platform/barefoot/sonic-platform-modules-arista +++ b/platform/barefoot/sonic-platform-modules-arista @@ -1 +1 @@ -Subproject commit 7304dbebdd80d7278abc6136e1df73190d405483 +Subproject commit cf36e513e41cd10a6d6aad5bcf71ef7871a2806d diff --git a/platform/broadcom/sonic-platform-modules-arista b/platform/broadcom/sonic-platform-modules-arista index 7304dbebdd..cf36e513e4 160000 --- a/platform/broadcom/sonic-platform-modules-arista +++ b/platform/broadcom/sonic-platform-modules-arista @@ -1 +1 @@ -Subproject commit 7304dbebdd80d7278abc6136e1df73190d405483 +Subproject commit cf36e513e41cd10a6d6aad5bcf71ef7871a2806d