Skip to content

[DPB] [Mellanox] added capability files for SN4700 platform #6278

Merged
lguohan merged 1 commit intosonic-net:masterfrom
vadymhlushko-mlnx:sn4700_dpb_capability_speeds_fix
Jan 25, 2021
Merged

[DPB] [Mellanox] added capability files for SN4700 platform #6278
lguohan merged 1 commit intosonic-net:masterfrom
vadymhlushko-mlnx:sn4700_dpb_capability_speeds_fix

Conversation

@vadymhlushko-mlnx
Copy link
Contributor

@vadymhlushko-mlnx vadymhlushko-mlnx commented Dec 23, 2020

Signed-off-by: Vadym Hlushko [email protected]

- Why I did it

platform.json and hwsku.json files has not a full set of speeds for split modes

- How I did it

Extended set of speeds for split modes for SN4700 platform

- How to verify it

Manually run DPB CLI commands

- 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)

@vadymhlushko-mlnx
Copy link
Contributor Author

retest this please

@vadymhlushko-mlnx
Copy link
Contributor Author

Please don't merge this PR until [DPB][MLNX][YANG-models] extended regex pattern according to MLNX systems speeds requirements PR will be merged

@vadymhlushko-mlnx vadymhlushko-mlnx marked this pull request as ready for review December 23, 2020 13:00
@liat-grozovik
Copy link
Collaborator

retest vs please

@liat-grozovik
Copy link
Collaborator

retest vsimage please

@liat-grozovik liat-grozovik changed the title [DPB][SN4700] extended set of speeds for split modes [DPB] [Mellanox] added capability files for SN4700 platform Jan 13, 2021
@liat-grozovik
Copy link
Collaborator

retest vsimage please

@liat-grozovik
Copy link
Collaborator

@lguohan can you please merge it? the azure build is stuck for few days

@lguohan lguohan merged commit 709c1ec into sonic-net:master Jan 25, 2021
@vadymhlushko-mlnx vadymhlushko-mlnx deleted the sn4700_dpb_capability_speeds_fix branch January 25, 2021 08:51
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