Skip to content

Remove S2 port mappings for 7060X O128 port skus#19094

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
dayouliu1:fix-7060X-128-etpx
Jul 3, 2025
Merged

Remove S2 port mappings for 7060X O128 port skus#19094
StormLiangMS merged 1 commit intosonic-net:masterfrom
dayouliu1:fix-7060X-128-etpx

Conversation

@dayouliu1
Copy link
Copy Markdown
Contributor

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
  • 202412

Approach

What is the motivation for this PR?

Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

How did you do it?

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).

@dayouliu1 dayouliu1 mentioned this pull request Jun 19, 2025
11 tasks
@dayouliu1 dayouliu1 force-pushed the fix-7060X-128-etpx branch from 029a246 to 99b9f69 Compare June 29, 2025 03:20
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@dayouliu1 dayouliu1 changed the title Remove extra unneeded port mappings for 7060X O128 port skus Remove S2 port mappings for 7060X O128 port skus Jun 29, 2025
@StormLiangMS
Copy link
Copy Markdown
Collaborator

hi @dayouliu1 conflict need to resolve.

@StormLiangMS StormLiangMS requested review from r12f and sdszhang July 2, 2025 08:28
Copy link
Copy Markdown
Contributor

@sdszhang sdszhang left a comment

Choose a reason for hiding this comment

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

lgtm

@dayouliu1 dayouliu1 force-pushed the fix-7060X-128-etpx branch from 99b9f69 to 65737a1 Compare July 2, 2025 20:40
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@dayouliu1
Copy link
Copy Markdown
Contributor Author

Resolved conflict

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
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 47afc24 into sonic-net:master Jul 3, 2025
18 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 3, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19394

mssonicbld pushed a commit that referenced this pull request Jul 4, 2025
Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
r12f added a commit to Azure/sonic-mgmt.msft that referenced this pull request Jul 9, 2025
…514)

Sync PR #19094 from sonic-net/sonic-mgmt to 202412.
Original PR: sonic-net/sonic-mgmt#19094

Co-authored-by: Dayou Liu <[email protected]>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

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
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed

Signed-off-by: Yael Tzur <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
…et#19094)

Approach
What is the motivation for this PR?
Remove Ethernet512, Ethernet513 mapping for 7060X 128 port skus as they are not needed
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