Skip to content

Commit 66fb4bc

Browse files
JibinBaodcaugher
authored andcommitted
Fix test_system_health issue (sonic-net#21216)
1. Only when dut is mellanox, we can call get_platform_data to get platform info 2. Relevant issue: sonic-net#21183
1 parent c868b3b commit 66fb4bc

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

tests/common/fixtures/duthost_utils.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -551,8 +551,11 @@ def is_support_fan(duthosts, rand_one_dut_hostname):
551551
Check if dut has fan
552552
"""
553553
duthost = duthosts[rand_one_dut_hostname]
554-
platform_data = get_platform_data(duthost)
555-
return platform_data['fans']['number'] > 0
554+
if is_mellanox_device(duthost):
555+
platform_data = get_platform_data(duthost)
556+
return platform_data['fans']['number'] > 0
557+
else:
558+
return True
556559

557560

558561
@pytest.fixture(scope='module')
@@ -561,8 +564,11 @@ def is_support_psu(duthosts, rand_one_dut_hostname):
561564
Check if dut has psu
562565
"""
563566
duthost = duthosts[rand_one_dut_hostname]
564-
platform_data = get_platform_data(duthost)
565-
return platform_data['psus']['number'] > 0
567+
if is_mellanox_device(duthost):
568+
platform_data = get_platform_data(duthost)
569+
return platform_data['psus']['number'] > 0
570+
else:
571+
return True
566572

567573

568574
def separated_dscp_to_tc_map_on_uplink(dut_qos_maps_module):

0 commit comments

Comments
 (0)