From 036a02fec017111f358b874fcbd64c106967e472 Mon Sep 17 00:00:00 2001 From: Algis Dumbris Date: Tue, 6 Oct 2026 10:55:07 +0300 Subject: [PATCH] ci(e2e): raise the Windows unit-test step timeout to 40m to match unit-tests.yml internal/server under -race on windows-latest now takes 650s to over 1200s, and the E2E workflow's 20m budget timed out on 6 of ~10 recent main runs with no hang in the goroutine dumps. unit-tests.yml already moved its Windows sweep to 40m for the same growth; bring this step back in step. --- .github/workflows/e2e-tests.yml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index ef506a2f7..be2e11f13 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -97,7 +97,7 @@ jobs: # they run unguarded in the main-only stress-tests job below so coverage # is preserved. # - # Timeout is 20m to match the Windows sweep in unit-tests.yml. It was + # Timeout matches the Windows sweep in unit-tests.yml. It was # 4m, which the windows-latest runner could not meet: internal/server # took 240s and died at `panic: test timed out after 4m0s` — with a # single test only 2s in, so cumulative slowness (Bleve index creation @@ -106,8 +106,12 @@ jobs: # 10m on 2026-09-12 (a74a7cf0f: 449s one push, 600s+ the next, a 4s # test in flight), so both budgets moved to 20m. Keep them in step; # the same code timing out in one and passing in the other is the - # signature of this bug returning. - run: go test -v -short -race -timeout 20m -skip "Binary|MCP|E2E|TestInfoEndpoint|TestGracefulShutdownNoPanic|TestSocketInfoEndpoint" ./internal/... + # signature of this bug returning. unit-tests.yml moved its Windows + # sweep to 40m when internal/server reached ~1100-1200s; this step was + # left at 20m and hit it on 6 of ~10 windows-latest main runs in + # 2026-10 (internal/server at 650s to over 1200s, no hang in the + # goroutine dumps), so it is 40m again to match. + run: go test -v -short -race -timeout 40m -skip "Binary|MCP|E2E|TestInfoEndpoint|TestGracefulShutdownNoPanic|TestSocketInfoEndpoint" ./internal/... # Binary tests are too flaky in CI due to server startup timing issues # All Binary tests consistently timeout waiting for server to be ready