Skip to content

[Mellanox] Adding SKU Mellanox-SN5600-C256#58

Closed
DavidZagury wants to merge 10 commits intomasterfrom
master-sn5600-c256
Closed

[Mellanox] Adding SKU Mellanox-SN5600-C256#58
DavidZagury wants to merge 10 commits intomasterfrom
master-sn5600-c256

Conversation

@DavidZagury
Copy link
Copy Markdown
Owner

@DavidZagury DavidZagury commented Jul 15, 2024

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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)

@dprital
Copy link
Copy Markdown
Collaborator

dprital commented Jul 15, 2024

@DavidZagury , Since CMIS Host Management should be enabled on this SKU, you need to add the following files as well:

  • optict_si_settings.json
  • media_settings.json

@DavidZagury
Copy link
Copy Markdown
Owner Author

@DavidZagury , Since CMIS Host Management should be enabled on this SKU, you need to add the following files as well:

  • optict_si_settings.json
  • media_settings.json

@dprital Resolved

@DavidZagury DavidZagury requested a review from stephenxs July 17, 2024 12:02
{% set default_cable = '40m' %}
{% set ingress_lossless_pool_size = '104773632' %}
{% set ingress_lossless_pool_xoff = '26165248' %}
{% set egress_lossless_pool_size = '162027520' %}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this value is too large to be accepted by SDK. Let's use 158229504

{% set default_cable = '300m' %}
{% set ingress_lossless_pool_size = '124303360' %}
{% set ingress_lossless_pool_xoff = '20529152' %}
{% set egress_lossless_pool_size = '162027520' %}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Let's use 158229504

@DavidZagury DavidZagury closed this Aug 8, 2024
DavidZagury pushed a commit that referenced this pull request Apr 14, 2025
…ly (sonic-net#22245)

#### Why I did it
src/sonic-stp
```
* 8320909 - (HEAD -> master, origin/master, origin/HEAD) MSTP configuration handling - Initialization APIS (#58) (14 hours ago) [Divya Kumaran Chandralekha]
```
#### How I did it
#### How to verify it
#### Description for the changelog
DavidZagury pushed a commit that referenced this pull request Apr 28, 2025
… automatically (sonic-net#832)

#### Why I did it
src/sonic-platform-common
```
* a3add6b - (HEAD -> 202412, origin/202412) [code sync] Merge code from sonic-net/sonic-platform-common:202411 to 202412 (#58) (21 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
DavidZagury pushed a commit that referenced this pull request Apr 28, 2025
…lly (sonic-net#861)

#### Why I did it
src/sonic-swss
```
* 2b334c7 - (HEAD -> 202412, origin/202412) [code sync] Merge code from sonic-net/sonic-swss:202411 to 202412 (#58) (21 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
DavidZagury pushed a commit that referenced this pull request May 22, 2025
…ly (sonic-net#21342)

#### Why I did it
src/dhcprelay
```
* b4f59d0 - (HEAD -> 202311, origin/202311) [202311] Install corresponding dependency libs from 2002311 for PR check (#58) (11 hours ago) [Yaqiang Zhu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
DavidZagury pushed a commit that referenced this pull request Mar 15, 2026
…tically (sonic-net#25573)

#### Why I did it
src/sonic-dash-api
```
* 63d9cb2 - (HEAD -> 202511, origin/202511) [action] [PR:56] Add list of trusted VNIs (#58) (8 hours ago) [mssonicbld]
* 694ce08 - [action] [PR:54] update docker slave name and switch to libboost1.83 (#60) (26 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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