Skip to content

Commit cef0a6f

Browse files
authored
Merge pull request sonic-net#154 from vperumal/skip_temp_cisco
Temporarily skipping few testcases in platform_tests/api/test_thermal.py and test_thermal_state_db.py for cisco devices Description of PR Due to a Jira, temporarily skipping few testcases in platform_tests/api/test_thermal.py and test_thermal_state_db.py. Will revert it as soon as the jira is fixed Approach What is the motivation for this PR? Due to a Jira, temporarily skipping few testcases in platform_tests/api/test_thermal.py and test_thermal_state_db.py. Will revert it as soon as the jira is fixed How did you do it? Skip in conditional_mark file SKIPPED [9] platform_tests/api/test_thermal.py: Unsupported platform API in mellanox. Skip in case of Cisco platform for T2 profile due to temperarory bug. Any platform specific information? Cisco 8808 chassis and T2 profile co-authorized by: [email protected]
2 parents 71e2b77 + 608bb49 commit cef0a6f

1 file changed

Lines changed: 19 additions & 3 deletions

File tree

tests/common/plugins/conditional_mark/tests_mark_conditions_platform_tests.yaml

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -586,15 +586,19 @@ platform_tests/api/test_thermal.py::TestThermalApi::test_get_low_threshold:
586586

587587
platform_tests/api/test_thermal.py::TestThermalApi::test_get_maximum_recorded:
588588
skip:
589-
reason: "Unsupported platform API"
589+
conditions_logical_operator: or
590+
reason: "Unsupported platform API in mellanox. Skip in case of Cisco platform for T2 profile due to temperarory bug."
590591
conditions:
591592
- "asic_type in ['mellanox']"
593+
- "asic_type in ['cisco-8000'] and topo_type in ['t2']"
592594

593595
platform_tests/api/test_thermal.py::TestThermalApi::test_get_minimum_recorded:
594596
skip:
595-
reason: "Unsupported platform API"
597+
conditions_logical_operator: or
598+
reason: "Unsupported platform API in mellanox. Skip in case of Cisco platform for T2 profile due to temperarory bug."
596599
conditions:
597600
- "asic_type in ['mellanox']"
601+
- "asic_type in ['cisco-8000'] and topo_type in ['t2']"
598602

599603
platform_tests/api/test_thermal.py::TestThermalApi::test_get_model:
600604
# Hardware components that we use for our sensors does not have IDPROM to store model and serial number details.
@@ -626,9 +630,11 @@ platform_tests/api/test_thermal.py::TestThermalApi::test_get_status:
626630

627631
platform_tests/api/test_thermal.py::TestThermalApi::test_get_temperature:
628632
skip:
629-
reason: "Unsupported platform API"
633+
conditions_logical_operator: or
634+
reason: "Unsupported platform API in mellanox. Skip in case of Cisco platform for T2 profile due to temperarory bug."
630635
conditions:
631636
- "asic_type in ['mellanox']"
637+
- "asic_type in ['cisco-8000'] and topo_type in ['t2']"
632638

633639
platform_tests/api/test_thermal.py::TestThermalApi::test_set_high_threshold:
634640
skip:
@@ -1028,6 +1034,16 @@ platform_tests/test_service_warm_restart.py:
10281034
conditions:
10291035
- "https://github.com/sonic-net/sonic-mgmt/issues/10362"
10301036

1037+
#######################################
1038+
#####test_thermal_state_db.py #####
1039+
#######################################
1040+
1041+
platform_tests/test_thermal_state_db.py:
1042+
skip:
1043+
reason: "Skip in case of Cisco platform for T2 profile due to temperarory bug."
1044+
conditions:
1045+
- "asic_type in ['cisco-8000'] and topo_type in ['t2']"
1046+
10311047
#######################################
10321048
##### test_xcvr_info_in_db.py #####
10331049
#######################################

0 commit comments

Comments
 (0)