[Bug][kubectl-plugin] E2e test DeferCleanup in BeforeEach may cause namespace race condition

[chiayi]

Search before asking

• I searched the issues and found no similar issues.

KubeRay Component

Others

What happened + What you expected to happen

#2745 May be causing e2e tests to fail due to race condition. Within each Describe block, the namespace variable is shared. And since this variable is shared, the DeferCleanup may cause namespace to be "cleaned up" once one It block is complete.

The expected behavior would be that each It block has its own namespace variable.

Reproduction script

Running the e2e test multiple times.

Anything else

No response

Are you willing to submit a PR?

• Yes I am willing to submit a PR!

[chiayi]

It could also be possible that since the tests that are most commonly failing are ray job submit and ray session, there could be a port overlap issue.

[chiayi]

I think the simple fix for this is to make the tests Ordered so that they run one after another, since by default, I believe Ginkgo runs the tests in random order and if no decorator is set, it runs tests in parallel. The downside of this would be that the tests will most likely take longer to run.