Skip to content

update full isolated topo for C512S2 and C448O16#19211

Merged
StormLiangMS merged 2 commits intosonic-net:masterfrom
sdszhang:topo_512_full
Jun 30, 2025
Merged

update full isolated topo for C512S2 and C448O16#19211
StormLiangMS merged 2 commits intosonic-net:masterfrom
sdszhang:topo_512_full

Conversation

@sdszhang
Copy link
Contributor

@sdszhang sdszhang commented Jun 26, 2025

Description of PR

Summary:
Update full topology for both C512S2 and C448O16 hwsku.

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?

Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?

generated the topo again.

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sdszhang
Copy link
Contributor Author

Note: This generate_topo.py in this PR has dependencies on #19202 to generate correct files.

ansible/veos Outdated
- t1-isolated-d448u8-lag
- t1-isolated-d56u2
- t1-isolated-v6-d448u16
- t1-isolated-v6-d448u8-lag
Copy link
Collaborator

Choose a reason for hiding this comment

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

Only the first port is having lag, do you mind to update to u15-lag?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated the topo to use the lag on the first physical port: PortList(LagPort(12), 13, 16, 17, 44, 45, 48, 49)

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sdszhang sdszhang marked this pull request as ready for review June 27, 2025 08:25
Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit d60f3f4 into sonic-net:master Jun 30, 2025
18 checks passed
@mssonicbld
Copy link
Collaborator

@sdszhang PR conflicts with 202505 branch

mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 1, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #19314

@sdszhang
Copy link
Contributor Author

sdszhang commented Jul 3, 2025

manual 202412 cherry-pick PR: Azure/sonic-mgmt.msft#459

yejianquan pushed a commit that referenced this pull request Jul 3, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Co-authored-by: Dashuai Zhang <[email protected]>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.
@sdszhang sdszhang deleted the topo_512_full branch September 13, 2025 10:21
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
What is the motivation for this PR?
Update the t0-isolated-xx and t1-isolated-xx full topo

How did you do it?
generated the topo again.

Signed-off-by: Yael Tzur <[email protected]>
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.

6 participants