Skip to content

[action] [PR:21228] [Mellanox]Removing FEC from port_config.ini for SN5600 SKUs (#21228)#21326

Merged
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/21228
Jan 16, 2025
Merged

[action] [PR:21228] [Mellanox]Removing FEC from port_config.ini for SN5600 SKUs (#21228)#21326
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/21228

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

  • Why I did it
    Removed FEC mode from port_config.ini for SN5600 SKUs. FEC mode needs to come as part of user configuration as it depends on peer end capabilities as well.

  • How I did it
    Modified port_config.ini

  • How to verify it
    Loading the SKU and testing default configuration generation

…t#21228)

- Why I did it
Removed FEC mode from port_config.ini for SN5600 SKUs. FEC mode needs to come as part of user configuration as it depends on peer end capabilities as well.

- How I did it
Modified port_config.ini

- How to verify it
Loading the SKU and testing default configuration generation
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #21228

@bingwang-ms
Copy link
Copy Markdown
Contributor

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 140efab into sonic-net:202405 Jan 16, 2025
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