Remove the Single tenant Broker

[matzew]

For release 0.16 we remove the single tenant broker.

Background

With the advent of the multitenant broker class in v0.14 of Eventing, we now have an easier to use and less resource consuming broker. The creation for the mt broker is mostly done via injection/labeling and applying CR files, backing different implementations. No special ClusterRoles are needed for the “non default” broker.

Problem

Single-tenant broker creation is much harder if not using the “default” broker, as stated here. While with the multi-tenant

Proposal

• set the MT broker as the new default, in the ConfigMap: 0.15 (see PR: Set MT based broker the default choice #3137)
• mark the ST broker as deprecated: 0.15
• update the knative/operator to reflect the new default config (see PR: With 0.15 the MTChannelBasedBroker is the new default broker operator#118)
• remove the ST broker (see PR: Remove channel based broker #3212)
• update the knative/docs repo to reflect the removal of the ChannelBasedBroker class (see PR: Remove ST Broker docs from master docs#2533)

[slinkydeveloper]

👍 for targeting 0.16

[matzew]

/assign @matzew

[vaikas]

I think we should also set the Broker status as deprecated with pointer to upgrade to other broker.
Something like this:
https://github.com/knative/eventing/pull/2374/files

[vaikas]

I'll remove the ChannelBasedBroker

[vaikas]

oops, just was going to comment, not close and comment :( :)

[matzew]

I keep it open util last two are merged.

doc and operator are pending

[matzew]

/close

all done

[knative-prow-robot]

@matzew: Closing this issue.

In response to this:

/close

all done

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.