Skip to content

Commit

Permalink
Skip tests/vxlan/test_vnet_bgp_route_precedence.py for non cisco and …
Browse files Browse the repository at this point in the history
…mnlx platforms (sonic-net#16266)

What is the motivation for this PR?
Skip tests/vxlan/test_vnet_bgp_route_precedence.py for non cisco and mnlx platforms

How did you do it?
Add skip condition in conditional mark file

How did you verify/test it?
Run tests/vxlan/test_vnet_bgp_route_precedence.py on broadcom platform
  • Loading branch information
ZhaohuiS authored Jan 2, 2025
1 parent 149fbfb commit 1d153c8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -2409,6 +2409,12 @@ vrf/test_vrf_attr.py::TestVrfAttrSrcMac::test_vrf1_neigh_with_default_router_mac
#######################################
##### vxlan #####
#######################################
vxlan/test_vnet_bgp_route_precedence.py:
skip:
reason: "test_vnet_bgp_route_precedence can only run on cisco and mnlx platforms."
conditions:
- "asic_type not in ['cisco-8000', 'mellanox']"

vxlan/test_vnet_route_leak.py:
skip:
reason: "Test skipped due to issue #8374"
Expand Down
2 changes: 1 addition & 1 deletion tests/vxlan/test_vnet_bgp_route_precedence.py
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ def fixture_setUp(duthosts,
data['loopback_v6'] = data['minigraph_facts']['minigraph_lo_interfaces'][0]['addr']
asic_type = duthosts[rand_one_dut_hostname].facts["asic_type"]
if asic_type not in ["cisco-8000", "mellanox"]:
raise RuntimeError("Pls update this script for your platform.")
pytest.skip(f"{asic_type} is not a supported platform for this test. Only support MNLX and CISCO platforms.")

# Should I keep the temporary files copied to DUT?
ecmp_utils.Constants['KEEP_TEMP_FILES'] = \
Expand Down

0 comments on commit 1d153c8

Please sign in to comment.