Skip to content

acceptance: parallelize tests and split connect/api-gateway into dedicated runners#5290

Open
kswap wants to merge 9 commits into
mainfrom
acceptance-test-parallelism
Open

acceptance: parallelize tests and split connect/api-gateway into dedicated runners#5290
kswap wants to merge 9 commits into
mainfrom
acceptance-test-parallelism

Conversation

@kswap
Copy link
Copy Markdown
Contributor

@kswap kswap commented May 5, 2026

Summary

  • Add t.Parallel() to all top-level test functions in the connect and api-gateway packages so tests within each package run concurrently instead of serially
  • Split connect_proxy_lifecycle_test.go into a new connect-lifecycle package and api_gateway_external_servers_test.go into a new api-gateway-external package, each with their own CI runner slot
  • Update all four CI matrix files (KinD, AKS, EKS, GKE) to add runner entries for the two new packages

Test plan

  • Verify CI checks pass on this PR (connect, connect-lifecycle, api-gateway, api-gateway-external runners all appear in Checks tab)
  • Confirm no test failures introduced by parallelism (unique release names via helpers.RandomName() ensure isolation)
  • Confirm the two new packages compile and run independently

@kswap kswap requested review from a team as code owners May 5, 2026 14:21
@kswap kswap added pr/no-changelog PR does not need a corresponding .changelog entry pr/no-backport signals that a PR will not contain a backport label labels May 5, 2026
@kswap kswap force-pushed the acceptance-test-parallelism branch from b9b0dd7 to c38050f Compare May 7, 2026 04:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-backport signals that a PR will not contain a backport label pr/no-changelog PR does not need a corresponding .changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant