Skip to content

Remove topo 'vs' in vxlan/test_vxlan_route_advertisement.py#20363

Merged
wangxin merged 1 commit intosonic-net:masterfrom
wangxin:vxlan-topo
Mar 6, 2026
Merged

Remove topo 'vs' in vxlan/test_vxlan_route_advertisement.py#20363
wangxin merged 1 commit intosonic-net:masterfrom
wangxin:vxlan-topo

Conversation

@wangxin
Copy link
Copy Markdown
Collaborator

@wangxin wangxin commented Aug 22, 2025

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

How did you do it?

Remove the "vs" topology from the topology mark in vxlan/test_vxlan_route_advertisement.py

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@StormLiangMS
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Copy Markdown
Collaborator Author

wangxin commented Sep 23, 2025

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin closed this Sep 23, 2025
@wangxin wangxin reopened this Sep 23, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Copy Markdown
Collaborator Author

wangxin commented Oct 23, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin closed this Oct 24, 2025
@wangxin wangxin reopened this Oct 24, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Copy Markdown
Collaborator Author

wangxin commented Nov 26, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Copy Markdown
Collaborator Author

wangxin commented Dec 1, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin merged commit 9ce0078 into sonic-net:master Mar 6, 2026
16 checks passed
ksravani-hcl pushed a commit to ksravani-hcl/sonic-mgmt that referenced this pull request Mar 10, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
ksravani-hcl pushed a commit to ksravani-hcl/sonic-mgmt that referenced this pull request Mar 10, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
aronovic pushed a commit to aronovic/sonic-mgmt that referenced this pull request Mar 10, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Mihut Aronovici <aronovic@cisco.com>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Mar 16, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Abhishek <abhishek@nexthop.ai>
vrajeshe pushed a commit to vrajeshe/sonic-mgmt that referenced this pull request Mar 23, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Venkata Gouri Rajesh Etla <vrajeshe@cisco.com>
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Apr 1, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
albertovillarreal-keys pushed a commit to albertovillarreal-keys/sonic-mgmt that referenced this pull request Apr 3, 2026
…t#20363)

"vs" is not really a valid topology. Remove it from the topology mark in vxlan/test_vxlan_route_advertisement.py.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants