Enhanced minigraph parser for packet-chasiss production graph #21255
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
via NETLINK RTA_PRIORITY field which can be used as attribute/metadata in fpmsyncd for different use-cases. Signed-off-by: Abhishek Dosi <[email protected]>
This reverts commit a279d8e.
…K RTA_PRIORITY field which can be used as attribute/metadata in fpmsyncd for different use-cases. Signed-off-by: Abhishek Dosi <[email protected]>
…a NETLINK RTA_PRIORITY field which can be used as attribute/metadata in fpmsyncd for different use-cases." This reverts commit 2776138.
Signed-off-by: Abhishek Dosi <[email protected]>
packet-chassis Signed-off-by: Abhishek Dosi <[email protected]>
abdosi
changed the title
|
@rlhui @yejianquan for viz. |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This reverts commit b78ad9b.
Signed-off-by: Abhishek Dosi <[email protected]>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@rlhui for viz. @qiluo-msft this is bug fix so we should be able to get exception on this. |
arlakshm
approved these changes
Dec 30, 2024
qiluo-msft
approved these changes
Jan 2, 2025
|
@rlhui : can you help with merge of this |
Javier-Tan
added
Approved for msft-202405 Branch
and removed
Request for msft-202405 Branch
labels
|
Cherry-pick PR to msft-202405: Azure/sonic-buildimage-msft#501 |
VladimirKuk
pushed a commit
to Marvell-switching/sonic-buildimage
that referenced
this pull request
Jan 21, 2025
…net#21255) Fixed 2 issue while parsing production minigraph: bfd flag not being set for STATIC_ROUTE table Different mac-address not getting generated for each ASIC How I fixed: In production switch_type is derived using ForwardingMethod attribute. Modified code to use chassis_type which is derived using ForwardingMethod Signed-off-by: Abhishek Dosi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What I did:
Fixed 2 issue while parsing production minigraph:
bfdflag not being set for STATIC_ROUTE tablemac-addressnot getting generated for each ASICHow I fixed:
In production
switch_typeis derived usingForwardingMethodattribute. Modified code to usechassis_typewhich is derived usingForwardingMethodHow I verify:
Added UT
Manual Verification.
MSFT ADO: 30791293