Skip to content

Skip test_bgp_queues test on Mellanox platform with t1-lag topology#12738

Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom
bingwang-ms:skip_bgp_queue_on_mlnx_t1_lag
May 14, 2024
Merged

Skip test_bgp_queues test on Mellanox platform with t1-lag topology#12738
bingwang-ms merged 1 commit intosonic-net:masterfrom
bingwang-ms:skip_bgp_queue_on_mlnx_t1_lag

Conversation

@bingwang-ms
Copy link
Collaborator

Description of PR

Summary:
The test case test_bgp_queues is failing on Mellanox platform with t1-lag topology because of known issue on router interface.
Earlier there was a PR #11705 to skip it, but that didn't work as expected as topo_type was used, which should be topo_name.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

The test case test_bgp_queues is failing on Mellanox platform with t1-lag topology.

How did you do it?

Update tests/common/plugins/conditional_mark/tests_mark_conditions.yaml

How did you verify/test it?

The change is verified by running on a Mellanox testbed with t1-lag topo.

collected 1 item                                                                                                                                                                                      

bgp/test_bgp_queue.py::test_bgp_queues[str2-msn2700-spy-2-None] SKIPPED (Unsupported topology or mellanox T1-LAG asic)                                                                          [100%]

Any platform specific information?

Mellanox platform specific.

Supported testbed topology if it's a new test case?

Not a new test case.

Documentation

@bingwang-ms bingwang-ms merged commit fc68e2b into sonic-net:master May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants