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

backport v1.9: Fix possible deadlock when querying netdevs #15227

Merged
merged 2 commits into from
Mar 8, 2021
Merged

backport v1.9: Fix possible deadlock when querying netdevs #15227

merged 2 commits into from
Mar 8, 2021

Commits on Mar 5, 2021

  1. vendor: Bump netlink to include timeouts 

    The bumped version includes the change [1] which sets a timeout for a
default netlink socket. This prevents deadlocks when receiving a netlink
msg (netlink is not reliable proto).

[1]: vishvananda/netlink#614

Signed-off-by: Martynas Pumputis <m@lambda.lt>
    @brb @christarazi
    brb authored and christarazi committed Mar 5, 2021
    Configuration menu
    Copy the full SHA
    1a1e861 View commit details
    Browse the repository at this point in the history

  2. node-neigh: Bump arping vsn to accept netlink.Link 

    This commit bumps github.com/cilium/arping version to accept
netlink.Link instead of net.Interface.

The change allows us to use netlink to query netdevs which avoids a
possible deadlock described in the previous commit.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
    @brb @christarazi
    brb authored and christarazi committed Mar 5, 2021
    Configuration menu
    Copy the full SHA
    57a80fd View commit details
    Browse the repository at this point in the history