Skip to content

Ported the fix https://github.com/FRRouting/frr/pull/18601 from FRR community#22408

Merged
yxieca merged 1 commit intosonic-net:masterfrom
sudhanshukumar22:mgmt_corrupted_candidate_ds
Jun 17, 2025
Merged

Ported the fix https://github.com/FRRouting/frr/pull/18601 from FRR community#22408
yxieca merged 1 commit intosonic-net:masterfrom
sudhanshukumar22:mgmt_corrupted_candidate_ds

Conversation

@sudhanshukumar22
Copy link
Contributor

Ported the fix FRRouting/frr#18601 from FRR community

Tested using static routes, save and reboot.
K>* 0.0.0.0/0 [0/0] via 10.59.128.1, eth0, 00:03:58
L * 1.1.1.1/32 is directly connected, Loopback0, 00:03:52
C>* 1.1.1.1/32 is directly connected, Loopback0, 00:03:52
C>* 10.59.128.0/20 is directly connected, eth0, 00:03:58
L> 10.59.142.159/32 is directly connected, eth0, 00:03:58
C>* 11.1.0.0/16 is directly connected, vsh0, 00:03:58
L> 11.1.22.238/32 is directly connected, vsh0, 00:03:58
C>* 20.1.1.0/24 is directly connected, Ethernet0, 00:02:22
L> 20.1.1.2/32 is directly connected, Ethernet0, 00:02:22
S>* 40.1.1.1/32 [1/0] is directly connected, Ethernet0, weight 1, 00:00:16
S>* 40.1.1.2/32 [1/0] is directly connected, Ethernet0, weight 1, 00:00:02

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

Tested branch (Please provide the tested image version)

FRR 10.0.1

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

yxieca
yxieca previously approved these changes Apr 22, 2025
@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@lipxu lipxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Built and verified the internal 202411 image with the PR. No issues.
Verify case:
1: Run the previous test_frr.py script, 50 iterations, no issue found.
2: Repeat running the script bgp/test_traffic_shift.py for more than 100 iterations, no issue found.
3: Repeat loading config from minigraph with TSA for more than 300 iterations, no issue found.

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator

@yxieca @StormLiangMS @prsunny Can we merge this fix?

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #23021

r12f added a commit to Azure/sonic-buildimage-msft that referenced this pull request Oct 1, 2025
…ity (#1675)

Sync PR #22408 from sonic-net/sonic-buildimage to 202412.
Original PR: sonic-net/sonic-buildimage#22408

Co-authored-by: sudhanshukumar22 <[email protected]>
r12f added a commit to Azure/sonic-buildimage-msft that referenced this pull request Oct 26, 2025
…ity (#1675)

Sync PR #22408 from sonic-net/sonic-buildimage to 202412.
Original PR: sonic-net/sonic-buildimage#22408

Co-authored-by: sudhanshukumar22 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants