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] Align Mellanox-SN4600C-D112C8 SKU with SKU definition #6817

Merged
merged 1 commit into from
Feb 18, 2021
Merged

[Mellanox] Align Mellanox-SN4600C-D112C8 SKU with SKU definition #6817

merged 1 commit into from
Feb 18, 2021

Conversation

shlomibitton
Copy link
Contributor

@shlomibitton shlomibitton commented Feb 18, 2021

Signed-off-by: Shlomi Bitton shlomibi@nvidia.com

- Why I did it
Mellanox-SN4600C-D112C8 SKU is not configured properly.
It should have 112 50G interfaces and 8 100G interfaces as described on this PR.

- How I did it
Modify sai_profile, port_config.ini and hwsku.json for DPB.

- How to verify it
Apply this HwSKU to a MSN4600C Mellanox platform.

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

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

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

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
@liat-grozovik
Copy link
Collaborator

@shlomibitton as we also need this to be in 202012 and in 202012 we don't have yet enabled DPB we will need the fixes related to port_config.ini and the sai xml to be available in 202012. once this is merged please have a seperate PR against 202012

@liat-grozovik liat-grozovik changed the title [Mellanox] Fix for Mellanox-SN4600C-D112C8 SKU [Mellanox] Align Mellanox-SN4600C-D112C8 SKU with SKU definition Feb 18, 2021
@liat-grozovik liat-grozovik merged commit 2076260 into sonic-net:master Feb 18, 2021
@liat-grozovik
Copy link
Collaborator

@daall can you please update 202012 with this PR?

@daall
Copy link
Contributor

daall commented Mar 10, 2021

@liat-grozovik there is a merge conflict, please submit a PR for 202012

@shlomibitton shlomibitton deleted the shlomi_Mellanox-SN4600C-D112C8_fix branch March 24, 2021 20:37
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
- Why I did it
Mellanox-SN4600C-D112C8 SKU is not configured properly.
It should have 112 50G interfaces and 8 100G interfaces as described on this PR.

- How I did it
Modify sai_profile, port_config.ini and hwsku.json for DPB.

- How to verify it
Apply this HwSKU to a MSN4600C Mellanox platform.

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants