Skip to content

[T2] Fix syntax error in prefix_list script#23481

Merged
yxieca merged 1 commit intosonic-net:masterfrom
yaqiangz:master_fix_anchor
Jul 29, 2025
Merged

[T2] Fix syntax error in prefix_list script#23481
yxieca merged 1 commit intosonic-net:masterfrom
yaqiangz:master_fix_anchor

Conversation

@yaqiangz
Copy link
Copy Markdown
Contributor

@yaqiangz yaqiangz commented Jul 25, 2025 •
edited
Loading

Why I did it

This PR introduce a syntax error #22337, then part missed in prefix_list script, which would cause syntax error

admin@sonic:~$ prefix_list status
/usr/bin/prefix_list: line 43: syntax error near unexpected token `fi'
/usr/bin/prefix_list: line 43: `    fi'
Work item tracking
  • Microsoft ADO (number only):

How I did it

Fix syntax error

How to verify it

Run cli and no error

admin@sonic:~$ prefix_list status
Operation is only supported on Upstream SpineRouter.

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@yaqiangz yaqiangz requested a review from lguohan as a code owner July 25, 2025 12:59
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Jul 25, 2025

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Jul 25, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@yaqiangz yaqiangz requested review from tjchadaga and removed request for lguohan July 25, 2025 13:00
@yxieca yxieca merged commit 275281b into sonic-net:master Jul 29, 2025
20 checks passed
@mssonicbld mssonicbld mentioned this pull request Jul 30, 2025
Merged
9 tasks
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Jul 30, 2025

Cherry-pick PR to 202505: #23522

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tjchadaga tjchadaga tjchadaga approved these changes

Assignees

No one assigned

Labels

Approved for 202505 Branch Included in 202505 Branch Request for 202505 Branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@yaqiangz @mssonicbld @tjchadaga @yejianquan @yxieca