Skip to content

[cherry-pick] [202311] Skip test_bgp_queues test on Mellanox platform with t1-lag topology#12739

Merged
bingwang-ms merged 1 commit intosonic-net:202311from
bingwang-ms:skip_bgp_queue_on_mlnx_t1_lag_202311
May 14, 2024
Merged

[cherry-pick] [202311] Skip test_bgp_queues test on Mellanox platform with t1-lag topology#12739
bingwang-ms merged 1 commit intosonic-net:202311from
bingwang-ms:skip_bgp_queue_on_mlnx_t1_lag_202311

Conversation

@bingwang-ms
Copy link
Collaborator

Description of PR

Summary:
This is a cherry-pick of PR #12738
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 4e10b4e into sonic-net:202311 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