]> git.puffer.fish Git - mirror/frr.git/commit
bgpd: Inconsistency in Local BGP GR state. 9043/head
authorprerana <prerana@vmware.com>
Fri, 9 Jul 2021 07:20:15 +0000 (00:20 -0700)
committermergify-bot <noreply@mergify.io>
Tue, 13 Jul 2021 15:41:14 +0000 (15:41 +0000)
commit7f2eeae2ed50e15d5adf6018bd6197e2e4388c92
tree81a9187c335c268930afbd0e273e93edde01a1d9
parent8b49607220d951ecb4f53b969e181f18ee95e4a2
bgpd: Inconsistency in Local BGP GR state.

Problem: Sometimes the configured Local GR state is not reflected in
show command and peer node. This is causing failures in few of the
BGP-GR topotests.

RCA: This problem is seen when the configuration of local GR state
happens when the BGP session is in OpenSent state and  moves to
Established after the configuration is complete.
When the session gets established, we move the GR state value from stub peer
to the config peer. This will result in overriding the GR state to
previous value.

Fix: The local GR state is modified only through CLI configuration and
does not change during BGP FSM transition. In this case it is not necessary
to transfer the GR state value from stub peer to config peer. This way we
can ensure that always the most recent config value is present in peer
datastructure.

Signed-off-by: Prerana-GB <prerana@vmware.com>
(cherry picked from commit 3f98a750c29bfce5e5d7ffa36fe5c0b1554d7b7a)
bgpd/bgp_fsm.c