From 4a008a7f1e66bdc147b2c63544509df7ba6b7dc6 Mon Sep 17 00:00:00 2001 From: mlsmaycon Date: Wed, 22 Jul 2026 05:13:01 +0000 Subject: [PATCH] Drop the harness lazy-connection override MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The nightly breakage this worked around is being fixed properly at the product level (lazy connection warmup, PR #6767), so the harness no longer needs to force eager peer connections. Until that PR merges, live e2e runs on this branch will time out in WaitProxyPeer again — re-run after it lands. --- e2e/harness/client.go | 5 ----- e2e/harness/proxy.go | 4 ---- 2 files changed, 9 deletions(-) diff --git a/e2e/harness/client.go b/e2e/harness/client.go index e31173374..19210349f 100644 --- a/e2e/harness/client.go +++ b/e2e/harness/client.go @@ -56,11 +56,6 @@ func StartClient(ctx context.Context, c *Combined, setupKey string) (*Client, er // Match the proxy: the combined relay is WebSocket-only, so the // client must use WS transport to keep a stable relay link to it. "NB_RELAY_TRANSPORT": "ws", - // Lazy connections defer the peer dial until traffic flows, which - // leaves `netbird status` at "0/N Connected" and starves - // WaitProxyPeer. Force eager connections so the proxy peer shows - // Connected as soon as the tunnel is up. - "NB_LAZY_CONN": "false", }, HostConfigModifier: func(hc *container.HostConfig) { hc.CapAdd = append(hc.CapAdd, "NET_ADMIN", "SYS_ADMIN", "SYS_RESOURCE") diff --git a/e2e/harness/proxy.go b/e2e/harness/proxy.go index 8d5b7e4f2..8db2c140f 100644 --- a/e2e/harness/proxy.go +++ b/e2e/harness/proxy.go @@ -77,10 +77,6 @@ func StartProxy(ctx context.Context, c *Combined, proxyToken string) (*Proxy, er // Management is plain HTTP in-cluster, so allow the proxy token to // ride a non-TLS gRPC connection. "NB_PROXY_ALLOW_INSECURE": "true", - // Keep the embedded overlay peer eager (see the client env note): - // lazy connections leave peers unconnected until traffic, which - // starves the harness's WaitProxyPeer. - "NB_LAZY_CONN": "false", // The combined server multiplexes the relay over WebSocket on :8080 // (no QUIC listener). The proxy's embedded relay client defaults to // QUIC, which fails here and flaps the relay link, churning the