add linkTraining support in minigraph parsing#26129
add linkTraining support in minigraph parsing#26129sdszhang wants to merge 3 commits intosonic-net:masterfrom
Conversation
Signed-off-by: Dashuai Zhang <[email protected]>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Pull request overview
Adds support in sonic-config-engine minigraph parsing to carry LinkTraining metadata from the minigraph into the generated PORT configuration, overriding any port_config.ini setting when present.
Changes:
- Parse
LinkTrainingfrom minigraphLinkMetadataproperties intolinkmetas - Override per-port
link_trainingin the generatedPORTtable based on minigraph values (similar to existingAutoNegotiationbehavior)
sdszhang
changed the title
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Dashuai Zhang <[email protected]>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
You can also share your feedback on Copilot code review. Take the survey.
Signed-off-by: Dashuai Zhang <[email protected]>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
minigraph.py is protected now. Is it possible to handle the new attribute in golden_config_db.json? |
|
closing this PR as minigraph is not suppose to be changed anymore. New method is using golden_config_db to generate link_training config. sonic-net/sonic-mgmt#23107 |
Why I did it
Add
LinkTrainingsupport in minigraph processing.Work item tracking
How I did it
parse 'LinkTraining' and convert to config_db.json.
How to verify it
verified locally.
Which release branch to backport (provide reason below if selected)
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)