Skip to content

[cherry-pick] Stabilize default-routes test and handle AZNG upstream LC route-map behavior#1037

Open
PriyanshTratiya wants to merge 1 commit into
Azure:202412from
PriyanshTratiya:Fix/default-route
Open

[cherry-pick] Stabilize default-routes test and handle AZNG upstream LC route-map behavior#1037
PriyanshTratiya wants to merge 1 commit into
Azure:202412from
PriyanshTratiya:Fix/default-route

Conversation

@PriyanshTratiya
Copy link
Copy Markdown

@PriyanshTratiya PriyanshTratiya commented Feb 23, 2026

Manual Cherry-Picking PRs: sonic-net/sonic-mgmt#15962 & sonic-net/sonic-mgmt#16532

This PR fixes issue sonic-net/sonic-mgmt#9052, where 'test_default_route_with_bgp_flap' fails with "Default route nexthops doesn't match the testbed topology" issue.

The count of the next hops is not matching with the upstream neighbor as per the topology (reading data from config facts).

In addition to the fix provided by sonic-net/sonic-mgmt#15962 for the issue sonic-net/sonic-mgmt#9052, this code change is also needed for handling AZNG route map changes on upstream LC.

Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
@sdszhang sdszhang changed the title manual cherry-picking PR#15962 and PR#16532 manual cherry-picking PR#15962 and PR#16532 to fix test_default_route failure Mar 11, 2026
@r12f
Copy link
Copy Markdown
Contributor

r12f commented Mar 12, 2026

let's update the title to show the original commit info.

@PriyanshTratiya PriyanshTratiya changed the title manual cherry-picking PR#15962 and PR#16532 to fix test_default_route failure [default-route] Stabilize default-route test and handle AZNG upstream LC route-map behavior Mar 12, 2026
@PriyanshTratiya PriyanshTratiya changed the title [default-route] Stabilize default-route test and handle AZNG upstream LC route-map behavior [dcherry-pick] Stabilize default-routes test and handle AZNG upstream LC route-map behavior Mar 12, 2026
@PriyanshTratiya PriyanshTratiya changed the title [dcherry-pick] Stabilize default-routes test and handle AZNG upstream LC route-map behavior [cherry-pick] Stabilize default-routes test and handle AZNG upstream LC route-map behavior Mar 12, 2026
@PriyanshTratiya
Copy link
Copy Markdown
Author

let's update the title to show the original commit info.

Updated, thanks Riff!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants