EnqueueRequestForOwner doesn't correctly enqueue cluster-scoped owner

[grantr]

Splitting out the cluster-scoped owner scenario from #214.

As pointed out by @liggitt, cluster-scoped owners are now explicitly supported.

EnqueueRequestForOwner doesn't support this relationship because it uses the namespace of the owned object as the namespace of the owner. We ran into this in knative/eventing#649.

[DirectXMan12]

/kind bug
/priority important-soon

[DirectXMan12]

/good-first-issue

[k8s-ci-robot]

@DirectXMan12:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

In response to this:

/good-first-issue

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.

[rajathagasthya]

I'd like to work on this, with some hand-holding. How do I get started with making changes?

cc @DirectXMan12

[DirectXMan12]

looks like #274 is working on this

[grantr]

I think this can be closed now that #274 is merged. Reopen if I'm wrong about that.
/close

[k8s-ci-robot]

@grantr: Closing this issue.

In response to this:

I think this can be closed now that #274 is merged. Reopen if I'm wrong about that.
/close

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.