tstest/natlab/{vmtest,vnet}, cmd/tta: add TestExitNode
Add a vmtest TestExitNode that brings up a client, two exit nodes, and a non-Tailscale webserver, each on its own NAT'd vnet network with a distinct WAN IP. The test cycles the client's exit node setting between off, exit1, and exit2 and asserts that the webserver echoes the expected post-NAT source IP for each. Three pieces were needed to make this work: vnet now forwards TCP between simulated networks at the packet level, mirroring the existing UDP path. When a guest VM sends TCP to another simulated network's WAN IP, the source network's gateway rewrites src via doNATOut and routeTCPPacket hands the packet off to the destination network, which rewrites dst via doNATIn and writes the rewritten frame onto the destination LAN. The TCP stacks of the two guest VM kernels talk end-to-end; vnet just NATs the IP/port headers in flight, so all TCP semantics (handshakes, options, sequence numbers, payload) are preserved without a gvisor TCP termination in the middle. Adds a focused TestInterNetworkTCP that exercises this path without any Tailscale machinery. cmd/tta binds its outbound dial to the default route's interface using SO_BINDTODEVICE. Without that, the moment tailscaled installs 0.0.0.0/0 → tailscale0 in response to setting an exit node, TTA's existing TCP connection to test-driver gets rerouted through the exit node. From the test driver's perspective the connection's packets then arrive with the exit node's WAN IP as the source rather than the client's, so they don't match the existing flow and the connection is dead — manifesting in the test as a hang on EditPrefs (which had actually completed in milliseconds on the daemon side, but whose response never made it back). Pinning the socket to the underlying NIC keeps TTA's agent connection on a real interface regardless of any policy routing tailscaled installs later. We bind rather than carry the Tailscale bypass fwmark because the fwmark approach is conditional on tailscaled having configured SO_MARK-based policy routing, while binding is unconditional. vmtest grows an Env.SetExitNode helper that sets ExitNodeIP via EditPrefs through the agent, used by the new test. Updates #13038 Change-Id: I9fc8f91848b7aa2297ef3eaf71fed9d96056a024 Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
This commit is contained in:
committed by
Brad Fitzpatrick
parent
10b63f27ce
commit
5c1738fd56
@@ -387,6 +387,45 @@ func (e *Env) startWebServer(ctx context.Context, n *Node) error {
|
||||
return nil
|
||||
}
|
||||
|
||||
// SetExitNode sets the client node's exit node to use for internet traffic.
|
||||
// If exitNode is nil, the client's exit node is cleared (i.e., turned off).
|
||||
// Otherwise exitNode must be a tailnet node with an approved 0.0.0.0/0 (and
|
||||
// ::/0) route, typically configured via [AdvertiseRoutes] and
|
||||
// [Env.ApproveRoutes].
|
||||
func (e *Env) SetExitNode(client, exitNode *Node) {
|
||||
e.t.Helper()
|
||||
ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
|
||||
defer cancel()
|
||||
|
||||
var ip netip.Addr
|
||||
if exitNode != nil {
|
||||
st, err := exitNode.agent.Status(ctx)
|
||||
if err != nil {
|
||||
e.t.Fatalf("SetExitNode: status for %s: %v", exitNode.name, err)
|
||||
}
|
||||
if len(st.Self.TailscaleIPs) == 0 {
|
||||
e.t.Fatalf("SetExitNode: %s has no Tailscale IPs", exitNode.name)
|
||||
}
|
||||
ip = st.Self.TailscaleIPs[0]
|
||||
}
|
||||
|
||||
if _, err := client.agent.EditPrefs(ctx, &ipn.MaskedPrefs{
|
||||
Prefs: ipn.Prefs{
|
||||
ExitNodeID: "",
|
||||
ExitNodeIP: ip,
|
||||
},
|
||||
ExitNodeIDSet: true,
|
||||
ExitNodeIPSet: true,
|
||||
}); err != nil {
|
||||
e.t.Fatalf("SetExitNode(%s -> %v): %v", client.name, exitNode, err)
|
||||
}
|
||||
if exitNode == nil {
|
||||
e.t.Logf("[%s] cleared exit node", client.name)
|
||||
} else {
|
||||
e.t.Logf("[%s] using exit node %s (%v)", client.name, exitNode.name, ip)
|
||||
}
|
||||
}
|
||||
|
||||
// ApproveRoutes tells the test control server to approve subnet routes
|
||||
// for the given node. The routes should be CIDR strings.
|
||||
func (e *Env) ApproveRoutes(n *Node, routes ...string) {
|
||||
|
||||
Reference in New Issue
Block a user