[action] [PR:21775] [chassis-packet] Allow Fallback Route to get programmed on Downstream LC#21833
Merged
mssonicbld merged 1 commit intosonic-net:202411from Apr 11, 2025
Merged
Conversation
What/Why I did: Initially I made the change that Fallback Route [routes from AH on Upstream LC] not to get programmed [mark them as deny] on Downstream LC's as reasoning was that on Downstream LC's can forward them based on Default-route. However that assumption is correct for example we have topology like this: RH is connected to ASIC0 of Upstream LC AH is connected to ASIC1 of Upstream LC Downstream LC will learn route from RH [including default-route] and will only forward to ASIC0 of upstream LC . Above assumption was fine if we always have AH and RH connected to same ASIC of Upstream LC. How I verify: UT updated Manual Verification.
Collaborator
Author
|
Original PR: #21775 |
Collaborator
Author
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
Author
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
DavidZagury
pushed a commit
to DavidZagury/sonic-buildimage
that referenced
this pull request
Apr 28, 2025
…-net#1052) Code sync sonic-net/sonic-buildimage:202411 => 202412 ``` * 903637b (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250424:0714 - Merge remote-tracking branch 'base/202411' into code-sync-202412 |\ | * 367199f (base/202411) mssonicbld 250423:1901 - [submodule] Update submodule sonic-swss to the latest HEAD automatically (sonic-net#22412) | * 0da8b10 mssonicbld 250422:1901 - [submodule] Update submodule sonic-sairedis to the latest HEAD automatically (sonic-net#22397) | * 4700d35 mssonicbld 250418:1901 - [submodule] Update submodule sonic-swss to the latest HEAD automatically (sonic-net#22364) | * 688b708 mssonicbld 250418:1901 - [CI]Add a pipeline to generates daily, successful virtual SONiC images (sonic-net#22368) | * c70a09c mssonicbld 250416:1601 - [submodule] Update submodule sonic-utilities to the latest HEAD automatically (sonic-net#22329) | * 1711475 mssonicbld 250416:0101 - [ci] Stop building slave docker for jessie and march (sonic-net#22331) | * 8f40740 Saikrishna Arcot 250414:1755 - [202411] Update to Linux 6.1.123 (sonic-net#21736) | * 2fd97fe mssonicbld 250411:1902 - [chassis-packet] Allow Fallback Route to get programmed on Downstream LC (sonic-net#21833) | * f2a76d5 Volodymyr Samotiy 250410:1845 - [202411][Mellanox] Integrate HW-MGMT Version 7.0040.2207 (sonic-net#22202) | * d9fd131 mssonicbld 250408:0920 - [submodule] Update submodule sonic-gnmi to the latest HEAD automatically (sonic-net#22258) | * bbd6e3c mssonicbld 250407:2200 - extend frr reconnect bmp retry interval from 1-2s into 10-15s. (sonic-net#22251) | * 3807de0 Aravind-Subbaroyan 250406:1803 - Update cisco-8000.ini (sonic-net#22247) | * bd3a419 Feng-msft 250407:1038 - Split frr_bmp feature switch for turn on FRR side bmp tunneling via Liquid (sonic-net#22243) | * b754560 mssonicbld 250404:1901 - [submodule] Update submodule sonic-utilities to the latest HEAD automatically (sonic-net#22232) | * fd4d058 zitingguo-ms 250404:0123 - [Broadcom] Upgrade xgs SAI to 12.3.6.2 (sonic-net#22219) | * 6042d69 mssonicbld 250404:0020 - [submodule] Update submodule sonic-sairedis to the latest HEAD automatically (sonic-net#22227) | * b9a0031 mssonicbld 250404:0020 - [submodule] Update submodule sonic-swss to the latest HEAD automatically (sonic-net#22228) | * 2bb570a sschlafman 250403:0854 - [202411] Changed SKU name to Mellanox-SN4280-C48-202411 (sonic-net#22121) | * a06b315 sschlafman 250403:0854 - [202411] Add new T1 Mellanox-SN4280-O8V40 SKU for 202411 (sonic-net#22108) | * 6921735 Ying Xie 250403:0716 - [FRR] fix FRR mgmtd losing configuration issue (sonic-net#22183) | * f7cbdd9 Stepan Blyshchak 250402:2200 - [orchagent.sh] mask SIGHUP before starting orchagent (sonic-net#22208) | * 4765324 sudhanshukumar22 250401:2234 - [202411][FRR] Port Fix from FRR community for issue sonic-net#18493 | * 874cbf4 Sai Kiran 250401:1546 - [docker-ptf] Port changes from master (sonic-net#22185) ``` --------- Co-authored-by: Sai Kiran <[email protected]> Co-authored-by: sudhanshukumar22 <[email protected]> Co-authored-by: Stepan Blyshchak <[email protected]> Co-authored-by: Ying Xie <[email protected]> Co-authored-by: sschlafman <[email protected]> Co-authored-by: mssonicbld <[email protected]> Co-authored-by: zitingguo-ms <[email protected]> Co-authored-by: Feng-msft <[email protected]> Co-authored-by: Aravind-Subbaroyan <[email protected]> Co-authored-by: Volodymyr Samotiy <[email protected]> Co-authored-by: Saikrishna Arcot <[email protected]>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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/Why I did:
Initially I made the change that Fallback Route [routes from AH on Upstream LC] not to get programmed [mark them as deny] on Downstream LC's as reasoning was that on Downstream LC's can forward them based on Default-route. However that assumption is correct for example we have topology like this:
RH is connected to ASIC0 of Upstream LC
AH is connected to ASIC1 of Upstream LC
Downstream LC will learn route from RH [including default-route] and will only forward to ASIC0 of upstream LC . Above assumption was fine if we always have AH and RH connected to same ASIC of Upstream LC.
How I verify:
UT updated
Manual Verification.