Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[routesync] Fix for stale dynamic neighbor #2553

Merged
merged 2 commits into from
Jan 12, 2023

Commits on Jan 6, 2023

  1. [routesync] Fix for stale dynamic neighbor

    Signed-off-by: vedganes <veda.ganesan@nokia.com>
    
    The changes are for fixing stale neighbor in the ASIC_DB and data path
    when eBGP neighbors are shutdown and neighbors are flushed. The problem
    is described in issue: sonic-net/sonic-buildimage#12442
    The root cause of this issue is due to not deleing the route from the
    ASIC_DB when the route's next hop is on eth0 or docker0 interface. The solution is
    to delete the route entry from ASIC_DB instead of just returning when the route's
    next hop is on the interface eth0 or docker0
    vganesan-nokia committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    a923a54 View commit details
    Browse the repository at this point in the history

Commits on Jan 10, 2023

  1. [routesync] Fix to handle warm restart

    Signed-off-by: vedganes <veda.ganesan@nokia.com>
    
    This commit fixes the warm restart unit test failure. When the route with
    only nh on eth0 or docker0 is removed and if the route is the default
    route, orchagent sends "add" black hole route to the syncd. So the ASIC
    DB gets n hset message. When this happens during warm restart, the unit
    test identifies this as unwanted setting and the unit test fails. To fix
    this issues, the route delete is sent only if the warm restart is not in
    progress. This is done following the same warm restart handling approach
    used for route delete in other palces.
    vganesan-nokia committed Jan 10, 2023
    Configuration menu
    Copy the full SHA
    e5f590c View commit details
    Browse the repository at this point in the history