MUX nbrs to use no-host-route & use nh update#3722
Open
manamand2020 wants to merge 25 commits intosonic-net:masterfrom
Open
MUX nbrs to use no-host-route & use nh update#3722manamand2020 wants to merge 25 commits intosonic-net:masterfrom
manamand2020 wants to merge 25 commits intosonic-net:masterfrom
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
6e74412 to
d81c7d4
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
d81c7d4 to
bf2bfdb
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
bf2bfdb to
72f9227
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
72f9227 to
d826c5c
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
d826c5c to
1a74366
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Manas Kumar Mandal <[email protected]>
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
1a74366 to
7352e2f
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
Signed-off-by: Manas Kumar Mandal <[email protected]>
manamand2020
force-pushed
the
dualtor_prefixrt_neigh
branch
from
c8ef06d to
8031243
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Manas Kumar Mandal <[email protected]>
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
|
/azpw run |
Collaborator
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
gordon-nexthop
pushed a commit
to nexthop-ai/sonic-utilities
that referenced
this pull request
Nov 25, 2025
What I did dualtor_neighbor_check.py script needs this change based on changes in the following PR: sonic-net/sonic-swss#3722 neighbors are no more deleted during switchover to standby
Collaborator
|
This PR cannot be merged as-is, the following are the gaps:
|
YairRaviv
pushed a commit
to YairRaviv/sonic-utilities
that referenced
this pull request
Jan 12, 2026
…rs (sonic-net#4007) What I did Mux neighbors now use prefix-based routes based on changes from the following PR: sonic-net/sonic-swss#3722 Presence of route prefix for neighbors does not mean tunnel-route use anymore. Show command now uses nexthop type to distinguish between tunnel and normal neighbor route. How to verify it Verified output on hardware. Signed-off-by: Manas Kumar Mandal <[email protected]>
YairRaviv
pushed a commit
to YairRaviv/sonic-utilities
that referenced
this pull request
Jan 12, 2026
What I did dualtor_neighbor_check.py script needs this change based on changes in the following PR: sonic-net/sonic-swss#3722 neighbors are no more deleted during switchover to standby
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
What I did
MUX neighbors will not use host route and full prefix route is explicitly created.
During mux state switchover only update the nexthop in these full prefix route of the neighbors.
Why I did it
This avoids multiple SAI calls done during MUX state switchover and reduces the switchover time.
How I verified it
Verified on hardware and VS Tests.
Details if related