Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix t1-isolated-d28u1 link mapping #16579

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sdszhang
Copy link
Contributor

@sdszhang sdszhang commented Jan 18, 2025

Description of PR

Summary:
In d28u1 topology, current link was not selected correctly for t1 hwsku.

Ethernet96  121,122,123,124     400G   9100    N/A  Ethernet13/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet100  125,126,127,128     400G   9100    N/A  Ethernet13/5  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet112               97     100G   9100     rs  Ethernet15/1  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet113               98     100G   9100     rs  Ethernet15/2  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet114               99     100G   9100     rs  Ethernet15/3  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet115              100     100G   9100     rs  Ethernet15/4  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet116              101     100G   9100     rs  Ethernet15/5  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet117              102     100G   9100     rs  Ethernet15/6  routed    down     down  OSFP 8X Pluggable Transceiver         off
Ethernet118              103     100G   9100     rs  Ethernet15/7  routed      up       up  OSFP 8X Pluggable Transceiver         off

After the fix:

admin@xxxx:~$ show interface status | grep up
  Ethernet0               17     100G   9100     rs   Ethernet1/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
 Ethernet16                9     100G   9100     rs   Ethernet3/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
 Ethernet32               57     100G   9100     rs   Ethernet5/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
 Ethernet48               33     100G   9100     rs   Ethernet7/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
 Ethernet64               89     100G   9100     rs   Ethernet9/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
 Ethernet80               65     100G   9100     rs  Ethernet11/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
 Ethernet96  121,122,123,124     400G   9100    N/A  Ethernet13/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet112               97     100G   9100     rs  Ethernet15/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet144              129     100G   9100     rs  Ethernet19/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet160              185     100G   9100     rs  Ethernet21/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet176              161     100G   9100     rs  Ethernet23/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet192              217     100G   9100     rs  Ethernet25/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet208              193     100G   9100     rs  Ethernet27/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet224              249     100G   9100     rs  Ethernet29/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet240              225     100G   9100     rs  Ethernet31/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet256              273     100G   9100     rs  Ethernet33/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet272              265     100G   9100     rs  Ethernet35/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet288              313     100G   9100     rs  Ethernet37/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet304              289     100G   9100     rs  Ethernet39/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet320              345     100G   9100     rs  Ethernet41/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet336              321     100G   9100     rs  Ethernet43/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet368              353     100G   9100     rs  Ethernet47/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet400              385     100G   9100     rs  Ethernet51/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet416              441     100G   9100     rs  Ethernet53/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet432              417     100G   9100     rs  Ethernet55/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet448              473     100G   9100     rs  Ethernet57/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet464              449     100G   9100     rs  Ethernet59/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet480              505     100G   9100     rs  Ethernet61/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
Ethernet496              481     100G   9100     rs  Ethernet63/1  routed      up       up  OSFP 8X Pluggable Transceiver         off
admin@xxx:~$

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202412

Approach

What is the motivation for this PR?

update topology with correct link

How did you do it?

update the check for current port

How did you verify/test it?

on physical testbed.

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@r12f r12f left a comment

Choose a reason for hiding this comment

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

thanks Shawn! signed off!

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.

3 participants