Sonic t2 support for radian #21732
Merged
Sonic t2 support for radian #21732
Conversation
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
Muckthebuck
requested review from
StormLiangMS,
qiluo-msft,
xumia and
lguohan
as code owners
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@Muckthebuck please look into the PR checker failure? |
src/sonic-bgpcfgd/tests/data/voq_chassis/policies.conf/param_base.json
Outdated
Show resolved
Hide resolved
Muckthebuck
force-pushed
the
SONiC-T2-support-for-RADIAN
branch
from
302775b to
0b08de0
Compare
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Muckthebuck
force-pushed
the
SONiC-T2-support-for-RADIAN
branch
from
0b08de0 to
9ce0f61
Compare
|
/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). |
Following files were altered or added: 1. Cli support to add a prefix - dockers/docker-fpm-fr/base_image_files/prefix_list - rules/docker-fpm-frr.mk 2. Manager to add appropriate prefix using jinja templates Signed-off-by: Mukul Chodhary <[email protected]>
Muckthebuck
force-pushed
the
SONiC-T2-support-for-RADIAN
branch
from
549cc00 to
3199d27
Compare
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Mukul Chodhary <[email protected]>
Muckthebuck
force-pushed
the
SONiC-T2-support-for-RADIAN
branch
from
3199d27 to
d214e49
Compare
Muckthebuck
force-pushed
the
SONiC-T2-support-for-RADIAN
branch
from
06869f2 to
949ebee
Compare
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Mukul Chodhary <[email protected]>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Mukul Chodhary <[email protected]>
|
/azp run Azure.sonic-buildimage |
replace IFS in help message with printf Signed-off-by: Mukul Chodhary <[email protected]>
|
/azp run Azure.sonic-buildimage |
tjchadaga
reviewed
Feb 21, 2025
dockers/docker-fpm-frr/frr/bgpd/templates/general/peer-group.conf.j2
Outdated
Show resolved
Hide resolved
tjchadaga
approved these changes
Feb 21, 2025
put the table map in correct spot for v4 Signed-off-by: Mukul Chodhary <[email protected]>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
rlhui
approved these changes
Feb 22, 2025
miatttao
pushed a commit
to miatttao/sonic-buildimage
that referenced
this pull request
Mar 5, 2025
Why I did it Adding support for RADIAN feature for SONiC T2 Work item tracking Microsoft ADO (number only):30112967 How I did it Cli commands to add/remove ANCHOR prefix to a PREFIX_LIST table in CONFIG_DB yang model changes for the new table PrefixListMgr to handle add/remove of configuration Templates add_radian/del_radian : to add or remove an anchor prefix list and aggregate address How to verify it Unit tests : config gen, manager and yang model --------- Signed-off-by: Mukul Chodhary <[email protected]>
prabhataravind
pushed a commit
to prabhataravind/sonic-buildimage
that referenced
this pull request
Mar 5, 2025
Why I did it Adding support for RADIAN feature for SONiC T2 Work item tracking Microsoft ADO (number only):30112967 How I did it Cli commands to add/remove ANCHOR prefix to a PREFIX_LIST table in CONFIG_DB yang model changes for the new table PrefixListMgr to handle add/remove of configuration Templates add_radian/del_radian : to add or remove an anchor prefix list and aggregate address How to verify it Unit tests : config gen, manager and yang model --------- Signed-off-by: Mukul Chodhary <[email protected]>
11 tasks
lguohan
pushed a commit
that referenced
this pull request
Mar 28, 2025
Why I did it fixes in T2 Radian Resiliency feature support checked in as part of PR #21732 Work item tracking Microsoft ADO 31982579: How I did it The changes take care of following Fix a typo which fetching chassis subtype in prefix_list script remove root permission validation during radian status check Input argument parsing fix during status check during add_radian/del_radian template rendering, prefix-list entry to have prefixlen passed as an argument, instead of hardcoding it How to verify it Execute the prefix_list script, and make sure add/del radian entries go through as expected. Ensure that status check doesnt require sudo Ensure that status check fails when passed second argument Ensure that backend code renders radian config as per expectation
11 tasks
yxieca
pushed a commit
that referenced
this pull request
Apr 9, 2025
Why I did it Radian T2 support was added by this PR #21732 We need to deny anchor routes from RH with ASNs in same DCs, hence need a attr to store asns in same DC Work item tracking Microsoft ADO (number only): 31979687 How I did it Add t2_group_asns in DEVICE_METADATA How to verify it UT passed
11 tasks
yxieca
pushed a commit
that referenced
this pull request
Apr 14, 2025
* [yang] Add anchor_route_source in DEVICE_METADATA * Update to listhy I did it T2 local anchor support was added by this PR #21732 We plan to use a new filed to specified the source of anchor route How I did it Add new field in DEVICE_METADAT How to verify it UT
yanjundeng
pushed a commit
to yanjundeng/sonic-buildimage
that referenced
this pull request
Apr 23, 2025
Why I did it Adding support for RADIAN feature for SONiC T2 Work item tracking Microsoft ADO (number only):30112967 How I did it Cli commands to add/remove ANCHOR prefix to a PREFIX_LIST table in CONFIG_DB yang model changes for the new table PrefixListMgr to handle add/remove of configuration Templates add_radian/del_radian : to add or remove an anchor prefix list and aggregate address How to verify it Unit tests : config gen, manager and yang model --------- Signed-off-by: Mukul Chodhary <[email protected]>
yanjundeng
pushed a commit
to yanjundeng/sonic-buildimage
that referenced
this pull request
Apr 23, 2025
Why I did it fixes in T2 Radian Resiliency feature support checked in as part of PR sonic-net#21732 Work item tracking Microsoft ADO 31982579: How I did it The changes take care of following Fix a typo which fetching chassis subtype in prefix_list script remove root permission validation during radian status check Input argument parsing fix during status check during add_radian/del_radian template rendering, prefix-list entry to have prefixlen passed as an argument, instead of hardcoding it How to verify it Execute the prefix_list script, and make sure add/del radian entries go through as expected. Ensure that status check doesnt require sudo Ensure that status check fails when passed second argument Ensure that backend code renders radian config as per expectation
yanjundeng
pushed a commit
to yanjundeng/sonic-buildimage
that referenced
this pull request
Apr 23, 2025
Why I did it Radian T2 support was added by this PR sonic-net#21732 We need to deny anchor routes from RH with ASNs in same DCs, hence need a attr to store asns in same DC Work item tracking Microsoft ADO (number only): 31979687 How I did it Add t2_group_asns in DEVICE_METADATA How to verify it UT passed
yanjundeng
pushed a commit
to yanjundeng/sonic-buildimage
that referenced
this pull request
Apr 23, 2025
* [yang] Add anchor_route_source in DEVICE_METADATA * Update to listhy I did it T2 local anchor support was added by this PR sonic-net#21732 We plan to use a new filed to specified the source of anchor route How I did it Add new field in DEVICE_METADAT How to verify it UT
vidyac86
pushed a commit
to vidyac86/sonic-buildimage
that referenced
this pull request
Apr 23, 2025
* [yang] Add anchor_route_source in DEVICE_METADATA * Update to listhy I did it T2 local anchor support was added by this PR sonic-net#21732 We plan to use a new filed to specified the source of anchor route How I did it Add new field in DEVICE_METADAT How to verify it UT
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.
Why I did it
Adding support for RADIAN feature for SONiC T2
Work item tracking
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Added support for RADIAN feature for SONiC T2
Link to config_db schema for YANG module changes
New table in config_db: PREFIX_LIST
Prefix list
A picture of a cute animal (not mandatory but encouraged)