-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[DPB] [Mellanox] added capability files for SN2700 platform #6003
[DPB] [Mellanox] added capability files for SN2700 platform #6003
Conversation
Signed-off-by: Vadym Hlushko <[email protected]>
Retest vsimage please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing none of the consecutive port in case of split 4
retest vsimage please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current implementation of SONiC DPB is not aligned with the HLD and is missing functionality required for SN2700 split to 4 (unable to remove consecutive port ) this option should be removed with a comment in the file.
comments are not allowed inside the JSON files |
…lementation of DPB doesn't support disabling of consecutive port Signed-off-by: Vadym Hlushko <[email protected]>
8c078cf
to
d1c6e4a
Compare
Please don't merge this PR until [DPB][MLNX][YANG-models] extended regex pattern according to MLNX systems speeds requirements PR will be merged |
- Why I did it
platform.json
andhwsku.json
files are required for a feature called Dynamic Port Breakout- How I did it
Created capability files according to platform specification
SN2700
- How to verify it
Full qualification requires bugs fixes reported under sonic-buildimage
- Which release branch to backport (provide reason below if selected)