Improve documentation on configuring the default BrokerClass

[matzew]

The doc https://knative.dev/docs/eventing/configuration/broker-configuration/#configuring-the-default-brokerclass-for-namespaces

is a little misleading or not fully correct.

• KafkaBroker -> there is no such class, the Kafka-based broker impl does use Kafka instead
• The config just contains the brokerClass field, usually each broker has different configuration, captured in a config map. This should be reflected in the doc as well. An example for some better Kafka config would go like:

apiVersion: operator.knative.dev/v1alpha1
kind: KnativeEventing
metadata:
  name: knative-eventing
  namespace: knative-eventing
spec:
  defaultBrokerClass: Kafka
  config:
    config-br-defaults:
      default-br-config: |
        clusterDefault:
          apiVersion: v1
          kind: ConfigMap
          name: kafka-broker-config
          namespace: knative-eventing

[github-actions[bot]]

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

[abrennan89]

RE apiVersion: operator.knative.dev/v1alpha1 - this isn't the same as the docs provided at the link because it's updating the eventing CR via the Operator rather than modifying the config map directly. IMO this needs to be updated here instead: https://knative.dev/docs/install/operator/configuring-eventing-cr/#configuring-the-default-broker-class

[github-actions[bot]]

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.