-
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][YANG-models] extended regex pattern according to Mellanox systems speeds requirements #6279
[DPB][YANG-models] extended regex pattern according to Mellanox systems speeds requirements #6279
Conversation
Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
retest vs please |
retest vsimage please |
retest vsimage please |
I think sonic-breakout_cfg.yang should be removed and backend should check the speed by platform.json, and not limit this into yang models. This is going to be accommodated by sonic-net/sonic-utilities#1020 and new PR(s). sonic-port.yang changes are fine. |
I believe this approach is better but it will take time until it will be available. Meanwhile we are blocked with many PRs. |
@zhenggen-xu , what are the new PRs you mentioned? |
OK, we can get this in, we will have new PR(s) to remove the sonic-breakout_cfg.yang and make the configMgmt logic to check the breakout mode instead. |
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.
Please add atleast a new test with it.
To add Test:
1.) Add Config at https://github.com/Azure/sonic-buildimage/blob/e52581e919c95e861cf630c9c7e1a840fc5e8ebd/src/sonic-yang-models/tests/yang_model_tests/yangTest.json#L2
2.) Add matching detail and expected error string at:
https://github.com/Azure/sonic-buildimage/blob/e52581e919c95e861cf630c9c7e1a840fc5e8ebd/src/sonic-yang-models/tests/yang_model_tests/test_yang_model.py#L52
Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
d71eb75
@praveen-li could you please review the last changes? |
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.
LGTM, note it may be possible that we may deprecate BRK_CFG Yang model later. Thx.
Retest vsimage please |
- Why I did it
All Mellanox platforms require DPB modes with a specific set of speeds example
- How I did it
Extended regex pattern inside YANG model.
Supported platforms:
SN2010
,SN2100
,SN2410
,SN2700
,SN3420
,SN3700
,SN3700C
,SN3800
,SN4600C
,SN4410
,SN4700
- How to verify it
Manually tested DPB CLI on all platform with all modes
- Which release branch to backport (provide reason below if selected)
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)