From c8c2b7fc45033d66fa58eb9d681345f1a41bff95 Mon Sep 17 00:00:00 2001 From: Kebo Liu Date: Tue, 8 Nov 2022 19:37:10 +0800 Subject: [PATCH] [Mellanox] [Platform API] Update SN2201 dynamic minimum fan speed table (#12602) - Why I did it Update SN2201 dynamic minimum fan speed table according to data provided by the thermal team. - How I did it Update the thermal table in device_data.py - How to verify it Run platform related regression Signed-off-by: Kebo Liu --- .../mellanox/mlnx-platform-api/sonic_platform/device_data.py | 4 ++++ platform/mellanox/mlnx-platform-api/tests/test_thermal.py | 4 ++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/platform/mellanox/mlnx-platform-api/sonic_platform/device_data.py b/platform/mellanox/mlnx-platform-api/sonic_platform/device_data.py index 94ed64d7d380..522287cdb531 100644 --- a/platform/mellanox/mlnx-platform-api/sonic_platform/device_data.py +++ b/platform/mellanox/mlnx-platform-api/sonic_platform/device_data.py @@ -156,6 +156,10 @@ }, 'x86_64-nvidia_sn2201-r0': { 'thermal': { + 'minimum_table': { + "unk_trust": {"-127:30": 13, "31:35": 14, "36:40": 15, "41:120": 16}, + "unk_untrust": {"-127:15": 13, "16:20": 14, "21:25": 15, "26:30": 16, "31:35": 17, "36:40": 18, "41:120": 19}, + }, "capability": { "comex_amb": False, "cpu_amb": True diff --git a/platform/mellanox/mlnx-platform-api/tests/test_thermal.py b/platform/mellanox/mlnx-platform-api/tests/test_thermal.py index a7fdc4d0bafe..63da97161d20 100644 --- a/platform/mellanox/mlnx-platform-api/tests/test_thermal.py +++ b/platform/mellanox/mlnx-platform-api/tests/test_thermal.py @@ -93,10 +93,10 @@ def test_chassis_thermal(self): assert gearbox_thermal_count == 2 assert cpu_thermal_count == 2 + @mock.patch('sonic_platform.device_data.DeviceDataManager.get_platform_name', mock.MagicMock(return_value='x86_64-nvidia_sn2201-r0')) + @mock.patch('sonic_platform.device_data.DeviceDataManager.get_thermal_capability', mock.MagicMock(return_value={'comex_amb': False, 'cpu_amb': True, 'swb_amb': True})) def test_chassis_thermal_includes(self): from sonic_platform.thermal import THERMAL_NAMING_RULE - DeviceDataManager.get_platform_name = mock.MagicMock(return_value='x86_64-nvidia_sn2201-r0') - DeviceDataManager.get_thermal_capability = mock.MagicMock(return_value={'comex_amb': False, 'cpu_amb': True, 'swb_amb': True}) chassis = Chassis() thermal_list = chassis.get_all_thermals() assert thermal_list