diff options
author | Constantine Peresypkin <constantine@caspiandb.com> | 2021-10-31 16:33:32 +0200 |
---|---|---|
committer | pkit <pconstantine@gmail.com> | 2021-11-02 10:43:56 +0200 |
commit | a0849e657836cc76fc94e09bcae0755944b46a5c (patch) | |
tree | 25052e45bbfb1781d2b7ab00fe7134e27e01e7cb /runsc/sandbox | |
parent | b822923b706d6d2c5206451040f51a8c2f961353 (diff) |
copy PERM ARP entries from namespace on boot
copy and setup PERMANENT (static) ARP entries
from CNI namespace to the sandbox
Fixes #3301
Diffstat (limited to 'runsc/sandbox')
-rw-r--r-- | runsc/sandbox/network.go | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/runsc/sandbox/network.go b/runsc/sandbox/network.go index 3451d1037..26aed6242 100644 --- a/runsc/sandbox/network.go +++ b/runsc/sandbox/network.go @@ -173,6 +173,23 @@ func createInterfacesAndRoutesFromNS(conn *urpc.Client, nsPath string, hardwareG continue } + // Collect data from the ARP table. + dump, err := netlink.NeighList(iface.Index, 0) + if err != nil { + return fmt.Errorf("fetching ARP table for %q: %w", iface.Name, err) + } + + var neighbors []boot.Neighbor + for _, n := range dump { + // There are only two "good" states NUD_PERMANENT and NUD_REACHABLE, + // but NUD_REACHABLE is fully dynamic and will be re-probed anyway. + if n.State == netlink.NUD_PERMANENT { + log.Debugf("Copying a static ARP entry: %+v %+v", n.IP, n.HardwareAddr) + // No flags are copied because Stack.AddStaticNeighbor does not support flags right now. + neighbors = append(neighbors, boot.Neighbor{IP: n.IP, HardwareAddr: n.HardwareAddr}) + } + } + // Scrape the routes before removing the address, since that // will remove the routes as well. routes, defv4, defv6, err := routesForIface(iface) @@ -203,6 +220,7 @@ func createInterfacesAndRoutesFromNS(conn *urpc.Client, nsPath string, hardwareG RXChecksumOffload: rxChecksumOffload, NumChannels: numNetworkChannels, QDisc: qDisc, + Neighbors: neighbors, } // Get the link for the interface. |