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

[Mellanox]Removing 8x DPB mode from SN5400 and SN5400_simx platforms #19265

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

dgsudharsan
Copy link
Collaborator

@dgsudharsan dgsudharsan commented Jun 10, 2024

Why I did it

Removed 8x mode from SN5400 and simx platforms as current DPB infrastructure doesn't support 8x mode with the subsequent port need to be disabled

Work item tracking
  • Microsoft ADO (number only):

How I did it

Updated platform.json of both devices.

How to verify it

Running show interfaces breakout command and verifying this mode doesn't exist.

{
    "Ethernet0": {
        "index": "1,1,1,1,1,1,1,1",
        "lanes": "0,1,2,3,4,5,6,7",
        "breakout_modes": {
            "1x400G[200G,100G,50G,40G,25G,10G]": [
                "etp1"
            ],
            "2x200G[100G,50G,40G,25G,10G]": [
                "etp1a",
                "etp1b"
            ],
            "4x100G[50G,25G,10G]": [
                "etp1a",
                "etp1b",
                "etp1c",
                "etp1d"
            ]
        },
        "default_brkout_mode": "1x400G[200G,100G,50G,40G,25G,10G]",
        "Current Breakout Mode": "1x400G[200G,100G,50G,40G,25G,10G]",
        "child ports": "Ethernet0",
        "child port speeds": "400G"
    },

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@dgsudharsan
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit 2960b76 into sonic-net:master Jul 1, 2024
20 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jul 25, 2024
…net#19265)

- Why I did it
Removed 8x mode from SN5400 and simx platforms as current DPB infrastructure doesn't support 8x mode with the subsequent port need to be disabled

- How I did it
Updated platform.json of both devices.

- How to verify it
Running show interfaces breakout command and verifying this mode doesn't exist.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #19693

mssonicbld pushed a commit that referenced this pull request Jul 25, 2024
- Why I did it
Removed 8x mode from SN5400 and simx platforms as current DPB infrastructure doesn't support 8x mode with the subsequent port need to be disabled

- How I did it
Updated platform.json of both devices.

- How to verify it
Running show interfaces breakout command and verifying this mode doesn't exist.
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
…net#19265)

- Why I did it
Removed 8x mode from SN5400 and simx platforms as current DPB infrastructure doesn't support 8x mode with the subsequent port need to be disabled

- How I did it
Updated platform.json of both devices.

- How to verify it
Running show interfaces breakout command and verifying this mode doesn't exist.
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Aug 1, 2024
…net#19265)

- Why I did it
Removed 8x mode from SN5400 and simx platforms as current DPB infrastructure doesn't support 8x mode with the subsequent port need to be disabled

- How I did it
Updated platform.json of both devices.

- How to verify it
Running show interfaces breakout command and verifying this mode doesn't exist.
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.

5 participants