Namespace scoped channel doesn't work

[pierDipi]

Describe the bug

Reconciling dispatcher RoleBinding failed: rolebindings.rbac.authorization.k8s.io \"imc-dispatcher\" is forbidden: user \"system:serviceaccount:knative-eventing:imc-controller\" (groups=[\"system:serviceaccounts\" \"system:serviceaccounts:knative-eventing\" \"system:authenticated\"]) is attempting to grant RBAC permissions not currently held:\n{APIGroups:[\"messaging.knative.dev\"], Resources:[\"inmemorychannels\"], Verbs:[\"patch\"]}\n{APIGroups:[\"messaging.knative.dev\"], Resources:[\"inmemorychannels/finalizers\"], Verbs:[\"patch\"]}\n{APIGroups:[\"messaging.knative.dev\"], Resources:[\"inmemorychannels/status\"], Verbs:[\"patch\"]}"

Expected behavior

It works.

To Reproduce

1. Apply:

apiVersion: messaging.knative.dev/v1
kind: InMemoryChannel
metadata:
  name: imc
  annotations:
    eventing.knative.dev/scope: namespace

2. Send events to it

Knative release version
>= 0.17

Additional context

• Reproducer as test Add KafkaSource to namespaced IMC test knative-extensions/eventing-kafka#314

[vaikas]

Hrm. We used to have finalizers on the IMC, but no longer require them, is this some left over cruft from that era?

[grantr]

This issue is still in v0.21.0. Should we move it to a future release milestone or to the backlog?

Also is it still critical-urgent if it's been unchanged for over a month? @pierDipi

[pierDipi]

It seems this was fixed by @lionelvillard in #4906.

/close

[knative-prow-robot]

@pierDipi: Closing this issue.

In response to this:

It seems this was fixed by @lionelvillard in #4906.

/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.