e2e test cleanup deletes resources it should not

[nirrozenbaum]

I was trying to run e2e test against an existing cluster I have which had some other resources in the "default" namespace.
the cleanup section of the e2e test suite deleted resources that were not created as part of the test.
IMO this is a very unexpected behavior and a major bug.
if ones wants to run e2e tests against his cluster, he should be able to do so without deleting non-related resources.

CC: @danehans

[ahg-g]

The test should create resources in a namespace and delete everything in that namespace.

[nirrozenbaum]

@ahg-g that’s correct. but as of today the test creates the resources in default namespace and then deletes all resources from default namespace, even those that were not created as part of the test.

[nirrozenbaum]

and according to this TODO written by @danehans -

gateway-api-inference-extension/test/e2e/epp/e2e_suite_test.go

Line 57 in 53cb18f

// TODO [danehans]: Must be "default" until https://github.com/kubernetes-sigs/gateway-api-inference-extension/issues/227 is fixed

the namespace cannot be changed in tests until issue #227 is fixed

[nirrozenbaum]

e2e tests were updated to support setting the namespace in #661.
closing this issue.