Skip to content

[M0-2VLAN] Update test_vlan_ping for m0-2vlan topo#15503

Merged
wangxin merged 1 commit intosonic-net:masterfrom
lizhijianrd:fix-test-vlan-ping-m0-2vlan
Nov 13, 2024
Merged

[M0-2VLAN] Update test_vlan_ping for m0-2vlan topo#15503
wangxin merged 1 commit intosonic-net:masterfrom
lizhijianrd:fix-test-vlan-ping-m0-2vlan

Conversation

@lizhijianrd
Copy link
Contributor

Description of PR

Summary:
Update test_vlan_ping for m0-2vlan topo.

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Update test_vlan_ping for m0-2vlan topo.

How did you do it?

Use topo_type instead of topo_name.

How did you verify/test it?

Verified on Nokia-7215 M0-2VLAN testbed.

vlan/test_vlan_ping.py::test_vlan_ping PASSED                                                                                   [100%]
============================================== 1 passed, 1 warning in 441.79s (0:07:21) ===============================================

Any platform specific information?

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

Documentation

Copy link
Contributor

@yaqiangz yaqiangz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wangxin wangxin merged commit 9ff0387 into sonic-net:master Nov 13, 2024
@lizhijianrd lizhijianrd deleted the fix-test-vlan-ping-m0-2vlan branch November 13, 2024 05:22
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Nov 13, 2024
What is the motivation for this PR?
Update test_vlan_ping for m0-2vlan topo.

How did you do it?
Use topo_type instead of topo_name.

How did you verify/test it?
Verified on Nokia-7215 M0-2VLAN testbed.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #15525

mssonicbld pushed a commit that referenced this pull request Nov 14, 2024
What is the motivation for this PR?
Update test_vlan_ping for m0-2vlan topo.

How did you do it?
Use topo_type instead of topo_name.

How did you verify/test it?
Verified on Nokia-7215 M0-2VLAN testbed.
sreejithsreekumaran pushed a commit to sreejithsreekumaran/sonic-mgmt that referenced this pull request Nov 15, 2024
What is the motivation for this PR?
Update test_vlan_ping for m0-2vlan topo.

How did you do it?
Use topo_type instead of topo_name.

How did you verify/test it?
Verified on Nokia-7215 M0-2VLAN testbed.
yutongzhang-microsoft pushed a commit to yutongzhang-microsoft/sonic-mgmt that referenced this pull request Nov 21, 2024
What is the motivation for this PR?
Update test_vlan_ping for m0-2vlan topo.

How did you do it?
Use topo_type instead of topo_name.

How did you verify/test it?
Verified on Nokia-7215 M0-2VLAN testbed.
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.

4 participants