Add drivers for bridge, macvlan, vlan, vxlan and add helpers to
LinkService to use them (SetMaster, RemoveMaster)
Signed-off-by: Jeroen Simonetti <jeroen@simonetti.nl>
* netns: remove iproute2 dependency
This commit introduces a breaking change to rtnetlink.NetNS.
The existing netns implementation had a few problems. It assumed that network
namespaces have names, that they would always be pinned to /var/run/netns, and
that numeric/integer references are pid references. This made the NetNS type
unusable for referring to existing netns by fd, such as ones created by other
libraries, or by opening procfs entries directly as demonstrated in the new
testutils.NetNS() function.
The forced dependency on the `ip` CLI tool also wasn't reasonable for a pure-Go
library. Using the old implementation in a scratch/distroless container would
quickly run into roadblocks.
This commit also removes the functionality of creating and pinning new netns.
There are plenty of options out in the Go ecosystem for that, and providing
your own is only a few lines of code.
Signed-off-by: Timo Beckers <timo@incline.eu>
* test: remove calls to unix.Setrlimit() in favor of rlimit.RemoveMemlock()
ebpf-go provides this out of the box and skips setting the rlimit on kernels
that support bpf memory cgroup accounting.
Signed-off-by: Timo Beckers <timo@incline.eu>
* neigh: fix flaky tests, add State field to Neigh entry
The flaky tests that were documented in the code are expected. Use the State
field to discard entries that can't reasonably be considered in tests.
Signed-off-by: Timo Beckers <timo@incline.eu>
* neigh: fix race in Conn.Neighbours
When running tests locally, I would frequently hit "too many/little matches,
expected 1, actual 0" due to other tests creating and deleting interfaces in
the common host netns used by all tests.
Neigh entries that fail the interface lookup can't have their Interface fields
populated and should be dropped from the result since the interface is no longer
there to begin with.
Signed-off-by: Timo Beckers <timo@incline.eu>
* xdp: refactor test suite to use test helpers and netns-driven tests
While running the test suite for testing netns-related changes, I noticed
some of the xdp tests started failing because they wanted to create a dummy
interface in the host network namespace.
Avoid the complexity of managing dummy interfaces altogether by running all
tests within their own netns and use the existing lo device that's present by
default.
Signed-off-by: Timo Beckers <timo@incline.eu>
* xdp,netkit: remove duplicate kernelMinReq in favor of testutils.SkipOnOldKernel
There were two implementations of this, so move them to common testutils.
Signed-off-by: Timo Beckers <timo@incline.eu>
---------
Signed-off-by: Timo Beckers <timo@incline.eu>
Add a GitHub actions workflow to run golanci-lint.
* Only uses latest Go version to avoid `gofmt` differences.
* Fixup linting issues.
* Ignore misspell `Prefered` as it is used as a public struct variable.
* Disable `unused-parameter` as it produces unecessary warning noise.
Signed-off-by: SuperQ <superq@gmail.com>
This PR will add the ability to use functional args for route
add/replace with rtnl. Currently, two functions are supported,
`WithRouteSrc` and `WithRouteAttrs`. This allows the consolidation of
the route methods, as well as adding the ability to BYO-Attributes for a
given route. This layout can of course be extended to add other
route-specific args, as well as a starting point for options and args for other parts of the
rtnl package as time goes on.
Note that this removes the `RouteAddSrc` and `RouteReplaceSrc`
functions.
Signed-off-by: Spencer Smith <robertspencersmith@gmail.com>
Change route scope decision to be based on gateway. This should
make more sense because a link without a gateway should reside
on the same subnet.
Signed-off-by: Brad Beam <brad.beam@talos-systems.com>
* rtnl: add ParseAddr and MustParseAddr helpers, add usage examples
Signed-off-by: Timo Beckers <timo@incline.eu>
* addr_*_test - test ParseAddr and MustParseAddr
Signed-off-by: Timo Beckers <timo@incline.eu>
* Examples: use first ParseCIDR return value as IP address
Signed-off-by: Timo Beckers <timo@incline.eu>
Added `RouteDel` to support route deletion.
Also added `RouteAddSrc` to support adding routes with a source parameter.
Signed-off-by: Andrea Barberio <insomniac@slackware.it>
