Conversation
link/netkit_test.go
Outdated
| blackhole := driver.NetkitPolicyDrop | ||
| err = conn.Link.New(&rtnetlink.LinkMessage{ | ||
| Family: unix.AF_UNSPEC, | ||
| Index: 1000, |
There was a problem hiding this comment.
This means we can't call the function more than once per test or concurrently.
Is there a way to get the kernel to allocate an ID + retrieve the ID from the netlink answer somehow?
There was a problem hiding this comment.
If we do not provide the index the kernel will assign the index, the same goes for the name, just like the ip command line tool without giving any arguments. We are getting the link with net.InterfaceByName so we do not need to provide the index. But even if we get rid of the index we still have a hard-coded interface name. I am not sure what we can do about that. If we do not give the name the kernel will assign names starting from nk0 and nk1, and whatever available next.
There was a problem hiding this comment.
Could we get the interface index from the calling test function? It would be the caller's responsibility to give a unique index for every call. Also, we would not need to call net.InterfaceByName nor return the index from net.InterfaceByName. Also, conn can come from the caller so that if we need to call the function more than once we would not need to create new connections every time. What do you think?
There was a problem hiding this comment.
Could we get the interface index from the calling test function? It would be the caller's responsibility to give a unique index for every call.
I like this, seems straightforward enough.
Also, conn can come from the caller so that if we need to call the function more than once we would not need to create new connections every time. What do you think?
That would make it more cumbersome to call (because we need another func to dial the conn). I'd just dial a new conn as needed until it turns out that doesn't work for some reason.
There was a problem hiding this comment.
Why not generate a random string at the start of the function and use it as the interface name?
There was a problem hiding this comment.
Can do, ultimately this isn't going to happen often I'd guess. So manually allocating indices is probably not too much of a hassle.
There was a problem hiding this comment.
Ended up adding a small sequence generator which allocates ids automatically.
brlbil
force-pushed
the
pr/brlbil/add_netkit_test
branch
from
9bf62a3 to
fab29fe
Compare
The netkit link support was added without a test suite due to a lack of support for driver-specific extractions in the `github.com/jsimonetti/rtnetlink` package. The driver support has been added to `rtnetlink`. This commit adds a test suite for the netkit link and pulls in the `github.com/jsimonetti/rtnetlink/v2` package for testing. Signed-off-by: Birol Bilgin <[email protected]>
lmb
force-pushed
the
pr/brlbil/add_netkit_test
branch
from
fab29fe to
45ae194
Compare
3 participants
The netkit link support was added without a test suite due to a lack of support for driver-specific extractions in the
github.com/jsimonetti/rtnetlinkpackage. The driver support has been added tortnetlink. This PR adds a test suite for the netkit link and pulls in thegithub.com/jsimonetti/rtnetlink/v2package for testing.Fixes: #1350