Skip to content

[Mellanox] Remove 2x40G from SN3800 hwsku#8712

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
dgsudharsan:3800_dpb_fix
Sep 13, 2021
Merged

[Mellanox] Remove 2x40G from SN3800 hwsku#8712
liat-grozovik merged 1 commit intosonic-net:masterfrom
dgsudharsan:3800_dpb_fix

Conversation

@dgsudharsan
Copy link
Collaborator

@dgsudharsan dgsudharsan commented Sep 9, 2021

Why I did it

Removed 2x40G for SN3800. This mode is not supported by hardware.

How I did it

Removing it from hwsku.json and platform.json

How to verify it

Load it in the device and check supported modes

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@yozhao101
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik changed the title [Mellanox]Removing 2x40G from S3800 [Mellanox] Remove 2x40G from SN3800 hwsku Sep 9, 2021
@dgsudharsan
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan
Copy link
Collaborator

lguohan commented Sep 11, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit d20f0cb into sonic-net:master Sep 13, 2021
judyjoseph pushed a commit that referenced this pull request Sep 14, 2021
- Why I did it
Removed 2x40G for SN3800. This mode is not supported by hardware.

- How I did it
Removing it from hwsku.json and platform.json

- How to verify it
Load it in the device and check supported modes
@dgsudharsan dgsudharsan deleted the 3800_dpb_fix branch March 9, 2023 01:50
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