🐛 Further cleanup to tmc-related e2e tests

[davidfestal]

Signed-off-by: David Festal dfestal@redhat.com

Summary

Further cleanup to tmc-related e2e tests:

• we should always add the CRDs for deployments, services, endpoints in the fake pclusters, since they are now all required by the Syncer to correctly work.
• In a Syncer Virtual Workspace test, we should have been waiting for the Syncer finalizer bing set before trying to patch the resource.
• Rewite of the Syncer virtual workspace tests to avoid starting 2 syncers when it's not necessary.
• Cleanup the way Syncer are started to sync user workspaces, and remove the need of the BoundWorkload method.

Additional fixes on code bugs found at this occasion:

• Skip the kcp-system in namespace and resource scheduling (was obviously bringing much load for nothing)
• Small fix in the workload sync command, to avoid adding a nil APIExport reference in the SupportedAPIExports
• ...

Related issue(s)

Required changes that should fix some flakes and race-condition-related bugs.

[davidfestal]

/test e2e-shared

[davidfestal]

/test shared

[openshift-ci]

@davidfestal: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

• /test boilerplate
• /test codegen
• /test deps
• /test e2e
• /test e2e-multiple-runs
• /test e2e-sharded
• /test e2e-shared
• /test images
• /test imports
• /test lint
• /test modules
• /test test

Use /test all to run all jobs.

In response to this:

/test shared

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.

[davidfestal]

/test e2e-shared

[davidfestal]

/test e2e-sharded

[jmprusi]

/lgtm

[davidfestal]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: davidfestal

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 [davidfestal]

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