Skip to content

Commit

Permalink
Move stable pfcwd tests from onboarding test set to PR test set (#16234)
Browse files Browse the repository at this point in the history
Some dataplane test scripts have been added to PR test and put into optional onboarding job to test case performance, I calculated the success rate of test scripts in recent 3 days, I think I can move high success rate test scripts to t0/t1 job now
  • Loading branch information
xwjiang-ms authored Dec 27, 2024
1 parent 02ca63d commit 503c220
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 8 deletions.
15 changes: 7 additions & 8 deletions .azure-pipelines/pr_test_scripts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,10 @@ t0:
- bgp/test_bgp_route_neigh_learning.py
- l2/test_l2_configure.py
- srv6/test_srv6_basic_sanity.py
- pfcwd/test_pfcwd_all_port_storm.py
- pfcwd/test_pfcwd_function.py
- pfcwd/test_pfcwd_timer_accuracy.py
- pfcwd/test_pfcwd_warm_reboot.py

t0-2vlans:
- dhcp_relay/test_dhcp_relay.py
Expand Down Expand Up @@ -439,6 +443,9 @@ t1-lag:
- ipfwd/test_nhop_group.py
- restapi/test_restapi_vxlan_ecmp.py
- srv6/test_srv6_basic_sanity.py
- pfcwd/test_pfcwd_all_port_storm.py
- pfcwd/test_pfcwd_function.py
- pfcwd/test_pfcwd_timer_accuracy.py

multi-asic-t1-lag:
- bgp/test_bgp_bbr.py
Expand Down Expand Up @@ -471,11 +478,6 @@ dpu:
- dash/test_dash_vnet.py

onboarding_t0:
# We will add a batch of T0 control plane cases and fix the failed cases later
- pfcwd/test_pfcwd_all_port_storm.py
- pfcwd/test_pfcwd_function.py
- pfcwd/test_pfcwd_timer_accuracy.py
- pfcwd/test_pfcwd_warm_reboot.py
# - platform_tests/test_advanced_reboot.py
- lldp/test_lldp_syncd.py
# Flaky, we will triage and fix it later, move to onboarding to unblock pr check
Expand All @@ -489,9 +491,6 @@ onboarding_t0:

onboarding_t1:
- pc/test_lag_member_forwarding.py
- pfcwd/test_pfcwd_all_port_storm.py
- pfcwd/test_pfcwd_function.py
- pfcwd/test_pfcwd_timer_accuracy.py
- bgp/test_bgp_bbr_default_state.py
- bgp/test_bgp_command.py
- generic_config_updater/test_mgmt_interface.py
Expand Down
10 changes: 10 additions & 0 deletions tests/common/plugins/conditional_mark/tests_mark_conditions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1549,6 +1549,16 @@ pfcwd/test_pfcwd_warm_reboot.py:
- "'dualtor' in topo_name"
- https://github.com/sonic-net/sonic-mgmt/issues/8400

pfcwd/test_pfcwd_warm_reboot.py::TestPfcwdWb::test_pfcwd_wb[async_storm:
skip:
reason: "Warm Reboot is not supported in T2 or in standalone topos. / Pfcwd tests skipped on m0/mx testbed. / Currently async_storm test is not supported on VS platform"
conditions_logical_operator: or
conditions:
- "'t2' in topo_name"
- "'standalone' in topo_name"
- "topo_type in ['m0', 'mx']"
- "asic_type in ['vs'] and https://github.com/sonic-net/sonic-mgmt/issues/16233"

#######################################
##### process_monitoring #####
#######################################
Expand Down

0 comments on commit 503c220

Please sign in to comment.