From a9f56e465ae58a27bf46f4973e34c6a30dbd4c8f Mon Sep 17 00:00:00 2001 From: shlomibitton <60430976+shlomibitton@users.noreply.github.com> Date: Tue, 28 Jul 2020 20:33:55 +0300 Subject: [PATCH] [Thermal] Fix for 'show platform fan' command (#71) When a fan is under\over speed, status should change to 'Not OK' along with red LED color. Signed-off-by: Shlomi Bitton --- sonic-thermalctld/scripts/thermalctld | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sonic-thermalctld/scripts/thermalctld b/sonic-thermalctld/scripts/thermalctld index f6e95a7d26ab..6d484a28e89d 100644 --- a/sonic-thermalctld/scripts/thermalctld +++ b/sonic-thermalctld/scripts/thermalctld @@ -316,7 +316,7 @@ class FanUpdater(object): ('drawer_name', drawer_name), ('model', str(try_get(fan.get_model))), ('serial', str(try_get(fan.get_serial))), - ('status', str(fan_fault_status)), + ('status', str(fan_fault_status and not(fan_status.under_speed or fan_status.over_speed))), ('direction', str(fan_direction)), ('speed', str(speed)), ('speed_tolerance', str(speed_tolerance)),