Skip to content

Commit 5f13b0c

Browse files
lolyuAharonMalkin
authored andcommitted
[dualtor] Skip warm/fast reboot cases on dualtor (sonic-net#19442)
Description of PR Summary: Fixes # (issue) Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Approach What is the motivation for this PR? Skip warm/fast reboot cases on dualtor. Signed-off-by: Longxiang Lyu [email protected] How did you do it? Add those cases to the conditional mark skip list. vrf.test_vrf.TestVrfWarmReboot.test_vrf_swss_warm_reboot vrf.test_vrf.TestVrfWarmReboot.test_vrf_system_warm_reboot gnmi.test_gnoi_system_reboot.test_gnoi_system_reboot_warm mvrf.test_mgmtvrf.TestReboot.test_warmboot mvrf.test_mgmtvrf.TestReboot.test_fastboot sflow.test_sflow.TestReboot.testFastreboot sflow.test_sflow.TestReboot.testWarmreboot How did you verify/test it? mvrf/test_mgmtvrf.py::TestReboot::test_fastboot SKIPPED (Dualtor topology doesn't support warm-reboot) [100%] mvrf/test_mgmtvrf.py::TestReboot::test_warmboot SKIPPED (Dualtor topology doesn't support warm-reboot) [100%] sflow/test_sflow.py::TestReboot::testFastreboot SKIPPED (Dualtor topology doesn't support warm-reboot) [100%] sflow/test_sflow.py::TestReboot::testWarmreboot SKIPPED (Dualtor topology doesn't support warm-reboot) [100%] vrf/test_vrf.py::TestVrfWarmReboot::test_vrf_swss_warm_reboot SKIPPED (Dualtor topology doesn't support warm-reboot) [100%] vrf/test_vrf.py::TestVrfWarmReboot::test_vrf_system_warm_reboot SKIPPED (Dualtor topology doesn't support warm-reboot) [100%] gnmi/test_gnoi_system_reboot.py::test_gnoi_system_reboot_warm SKIPPED (Warm reboot should only run on t0 topology but not on dualtor) [100%] Signed-off-by: Longxiang Lyu <[email protected]> Signed-off-by: Aharon Malkin <[email protected]>
1 parent d9d4e49 commit 5f13b0c

1 file changed

Lines changed: 39 additions & 1 deletion

File tree

tests/common/plugins/conditional_mark/tests_mark_conditions.yaml

Lines changed: 39 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1582,9 +1582,11 @@ gnmi/test_gnoi_killprocess.py:
15821582

15831583
gnmi/test_gnoi_system_reboot.py::test_gnoi_system_reboot_warm:
15841584
skip:
1585-
reason: "Warm reboot should only run on t0 topology"
1585+
reason: "Warm reboot should only run on t0 topology but not on dualtor"
1586+
conditions_logical_operator: or
15861587
conditions:
15871588
- "topo_type not in ['t0']"
1589+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
15881590

15891591
#######################################
15901592
##### hash #####
@@ -2080,6 +2082,18 @@ mvrf/test_mgmtvrf.py:
20802082
- "topo_type in ['m0', 'mx', 'm1']"
20812083
- "platform in ['x86_64-nokia_ixr7250e_36x400g-r0']"
20822084

2085+
mvrf/test_mgmtvrf.py::TestReboot::test_fastboot:
2086+
skip:
2087+
reason: "Dualtor topology doesn't support advanced-reboot"
2088+
conditions:
2089+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
2090+
2091+
mvrf/test_mgmtvrf.py::TestReboot::test_warmboot:
2092+
skip:
2093+
reason: "Dualtor topology doesn't support advanced-reboot"
2094+
conditions:
2095+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
2096+
20832097
#######################################
20842098
##### nat #####
20852099
#######################################
@@ -2710,6 +2724,18 @@ sflow/test_sflow.py:
27102724
conditions:
27112725
- "asic_type in ['vs']"
27122726

2727+
sflow/test_sflow.py::TestReboot::testFastreboot:
2728+
skip:
2729+
reason: "Dualtor topology doesn't support advanced-reboot"
2730+
conditions:
2731+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
2732+
2733+
sflow/test_sflow.py::TestReboot::testWarmreboot:
2734+
skip:
2735+
reason: "Dualtor topology doesn't support advanced-reboot"
2736+
conditions:
2737+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
2738+
27132739
#######################################
27142740
##### show_techsupport #####
27152741
#######################################
@@ -3169,6 +3195,18 @@ vrf/test_vrf.py::TestVrfAclRedirect:
31693195
- "'switch' in vars() and len([capabilities for capabilities in switch.values() if 'REDIRECT_ACTION' in capabilities]) == 0"
31703196
- "asic_type in ['vs']"
31713197

3198+
vrf/test_vrf.py::TestVrfWarmReboot::test_vrf_swss_warm_reboot:
3199+
skip:
3200+
reason: "Dualtor topology doesn't support advanced-reboot"
3201+
conditions:
3202+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
3203+
3204+
vrf/test_vrf.py::TestVrfWarmReboot::test_vrf_system_warm_reboot:
3205+
skip:
3206+
reason: "Dualtor topology doesn't support advanced-reboot"
3207+
conditions:
3208+
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56', 'dualtor-aa-64-breakout']"
3209+
31723210
vrf/test_vrf_attr.py:
31733211
skip:
31743212
reason: "Vrf tests are skipped in PR testing."

0 commit comments

Comments
 (0)