pimd: Fix PIM MLAG Update Peer Zebra Status Upon Local MLAG Connectio… #15918
Conversation
This file contains 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
routingrocks
force-pushed
the
rvaratharaj/pim_mlag_state_fix
branch
from
2e85117 to
b45ff52
Compare
routingrocks
force-pushed
the
rvaratharaj/pim_mlag_state_fix
branch
from
15dd631 to
dd5abf7
Compare
…n Restoration Issue: In scenarios where the local MLAG connection is down, we currently halt processing peer MLAG messages. However, upon restoration of the local connection, we fail to update the peer Zebra status accordingly. Consider the case where the peer is up and sends FRR status messages to the local node. If CLAGd restarts on the local node while FRR is running, the local CLAGd assumes the peer is still down even when it's up. Fix: Update the peer Zebra status once the local MLAG connection is restored Testing: UT Ticket: # Signed-off-by: Rajesh Varatharaj <[email protected]>
routingrocks
force-pushed
the
rvaratharaj/pim_mlag_state_fix
branch
from
dd5abf7 to
2eeafae
Compare
|
CI_RUN |
|
ci:rerun |
|
@routingrocks are you going to continue with this? |
I need to work on this, got fully occupied with the current tasks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
…n Restoration
Issue:
In scenarios where the local MLAG connection is down, we currently halt processing peer MLAG messages. However, upon restoration of the local connection,
we fail to update the peer Zebra status accordingly.
Consider the case where the peer is up and sends FRR status messages to the local node. If CLAGd restarts on the local node while FRR is running, the local CLAGd assumes the peer is still down even when it's up.
Fix:
Update the peer Zebra status once the local MLAG connection is restored
Testing: UT
Ticket: #
Signed-off-by: Rajesh Varatharaj [email protected]