Sonic t2 support for radian #21732
Merged
Sonic t2 support for radian #21732
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
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 <70460358+Muckthebuck@users.noreply.github.com>
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 <70460358+Muckthebuck@users.noreply.github.com>
Muckthebuck
force-pushed
the
SONiC-T2-support-for-RADIAN
branch
from
3199d27 to
d214e49
Compare
Signed-off-by: Mukul Chodhary <70460358+Muckthebuck@users.noreply.github.com>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@Muckthebuck can you look into the build failure? |
deepak-singhal0408
approved these changes
Feb 19, 2025
|
@rlhui, please help merge this PR? Thanks, |
tjchadaga
reviewed
Feb 20, 2025
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Mukul Chodhary <70460358+Muckthebuck@users.noreply.github.com>
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 <70460358+Muckthebuck@users.noreply.github.com>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Mukul Chodhary <70460358+Muckthebuck@users.noreply.github.com>
|
/azp run Azure.sonic-buildimage |
replace IFS in help message with printf Signed-off-by: Mukul Chodhary <70460358+Muckthebuck@users.noreply.github.com>
|
/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 <70460358+Muckthebuck@users.noreply.github.com>
|
/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 <70460358+Muckthebuck@users.noreply.github.com>
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 <70460358+Muckthebuck@users.noreply.github.com>
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)