append namespace to resources to allow parallel conformance tests

[xrstf]

This PR makes it possible to run hydrophone concurrently in the same cluster by appending its namespace to the cluster-wide resource names (e.g. the ClusterRole hydrophone becomes hydrophone:<namespace>). I also fixed some leftover hardcoded names and changed the cleanup logic to just delete the namespace, letting Kubernetes take care of garbage collection.

[k8s-ci-robot]

Hi @xrstf. Thanks for your PR.

I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[rjsadow]

/ok-to-test
/assign

[dims]

/assign @rjsadow @reetasingh

[reetasingh]

/lgtm

[rjsadow]

@xrstf this looks good to me. I would like to add a test for this, we can add a parameter in hack/run_e2e.sh to set a namespace then create a presubmit to run a simple test In a set namespace. If you'd like to update the script in this pr or another that would be great, if not I'm happy to get to it later.

/lgtm
/approve
/hold
Feel free to remove the hold when you're ready.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rjsadow, xrstf

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [rjsadow]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[xrstf]

@rjsadow I've added the NAMESPACE parameter to the script and rebased the PR. :) Will create the presubmit afterwards.

[rjsadow]

Will it be an issue if namespace is set to an empty string? I don't see any validation to set it to the default. Does Viper take care of this somehow?

[xrstf]

I followed the same logic we already have for the FOCUS/SKIP/DRY_RUN variables further down. Besides the kind stuff, I can run the run-e2e.sh locally just fine. Also, the presubmits in this PR succeeded, so it should be fine.

[rjsadow]

Sounds good, let's give it a shot.

[rjsadow]

/lgtm

[xrstf]

/hold cancel