Repository navigation
cleanup: re-enable parallel tests #361
Description
Activity
A summary of the changes CodeRabbit can apply:
-
Replace the shared WorkDir usage with a per-test temporary directory in test/e2e/framework/kcp.go (change wsKubeconfigPath from filepath.Join(WorkDir, ws.Name+".kubeconfig") to filepath.Join(t.TempDir(), ws.Name+".kubeconfig")) to eliminate kubeconfig file races and re-enable safe parallel tests.
-
Update test/e2e/framework/kcp.go to write the workspace kubeconfig into a per-test temporary directory by replacing WorkDir with t.TempDir() for wsKubeconfigPath, ensuring kubeconfig files are created in isolated temp locations.
- ✅ Create PR with these edits
- 📋 Get copyable edits
-
Discussed in planning, we need to recheck if this is still needed
- moved this from Backlog to In Progress in Platform Mesh & Kube Projects - Backlog
on Aug 26, 2026 I am trying to reproduce the issue, but when I check out the original merge commit (640871f) and re-enable the
t.Parallel(), the contrib-kcp tests pass consistently. I am runningrm -rf .kcp && make test-e2e-contrib-kcp(in more recent versions of kbind you can leave out thermsince it was made part of the Makefile).- moved this from In Progress to Done in Platform Mesh & Kube Projects - Backlog
on Sep 8, 2026 Actually this was me having an .env file that forced parallelism to 1. :D
- moved this from In Progress to Blocked/Waiting in Platform Mesh & Kube Projects - Backlog
on Sep 22, 2026
After #351 was merged, the parallel test code was left in a bit of a not-ideal state.
We should come back and clean it.
Look for the issue link in the code for more details.
The problem here was the after
bind logincontext switches, and in the parallel tests, this means - the overridden context.