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.

Sign up for GitHub

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

Jump to bottom

[bgpmon]: Fix exception in bgpmon caused by duplicate bgp neighbor ID #6546

Merged
merged 3 commits into from
Jan 27, 2021
Merged

[bgpmon]: Fix exception in bgpmon caused by duplicate bgp neighbor ID #6546

merged 3 commits into from
Jan 27, 2021

Commits on Jan 25, 2021

  1. Fix exception in bgpmon caused by duplicate keys 

    It is possible that BGP neighbors in IPv4 and IPv6 address families
share the same name (such as bgp monitor). However, such case is not
handled in bgpmon, and an Exception will be raised. This commit will
address the issue.

Signed-off-by: bingwang <bingwang@microsoft.com>
    @bingwang-ms
    bingwang-ms committed Jan 25, 2021
    Configuration menu
    Copy the full SHA
    a56753c View commit details
    Browse the repository at this point in the history

Commits on Jan 26, 2021

  1. Use set instead of list to avoid duplicate keys. 

    Signed-off-by: bingwang <bingwang@microsoft.com>
    @bingwang-ms
    bingwang-ms committed Jan 26, 2021
    Configuration menu
    Copy the full SHA
    412fd58 View commit details
    Browse the repository at this point in the history

  2. Fix typo in comment. 

    Signed-off-by: bingwang <bingwang@microsoft.com>
    @bingwang-ms
    bingwang-ms committed Jan 26, 2021
    Configuration menu
    Copy the full SHA
    d20cd84 View commit details
    Browse the repository at this point in the history