Skip to content

T2: anchor prefix fixes #22130

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

deepak-singhal0408
Copy link
Contributor

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

  1. Fix a typo which fetching chassis subtype in prefix_list script
  2. remove root permission validation during radian status check
  3. Input argument parsing fix during status check
  4. 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

  1. Execute the prefix_list script, and make sure add/del radian entries go through as expected.
  2. Ensure that status check doesnt require sudo
  3. Ensure that status check fails when passed second argument
  4. Ensure that backend code renders radian config as per expectation

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

202405-chassis

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@deepak-singhal0408 deepak-singhal0408 changed the title Deepsinghal/anchor prefix fixes T2: anchor prefix fixes Mar 25, 2025
@deepak-singhal0408 deepak-singhal0408 self-assigned this Mar 25, 2025
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@deepak-singhal0408 deepak-singhal0408 requested review from tjchadaga and removed request for lguohan and StormLiangMS March 26, 2025 06:12
@deepak-singhal0408
Copy link
Contributor Author

@yaqiangz for viz.

@lguohan lguohan merged commit 2ae5309 into sonic-net:master Mar 28, 2025
21 checks passed
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants