kyma-project / kyma

Kyma is an opinionated set of Kubernetes-based modular building blocks, including all necessary capabilities to develop and run enterprise-grade cloud-native applications.
https://kyma-project.io
Apache License 2.0
1.51k stars 404 forks source link

Test Kyma behavior with knative-kafka when reaching the namespace event hubs quota #7665

Closed devguyio closed 4 years ago

devguyio commented 4 years ago

Description

Azure Event Hub namespaces has a quota for Event Hubs. The behavior of Kyma that uses Knative-Kafka channel implementation is unknown when reaching that limit and we need to figure it out.

AC

montaro commented 4 years ago

After consuming the quota, creating a new app will have a NOT READY Channel

❯ kubectl -n kyma-integration get kafkachannels.knativekafka.kyma-project.io test-app-7
NAME          READY   REASON        HOSTNAME    AGE
test-app-7    False   TopicFailed               6h21md

Provisioning a namespace with an Event Service, will have a NOT READY Channel and Broker. The Broker and Trigger pods won't be created

❯ kubectl -n test-ns5 get kafkachannels.knativekafka.kyma-project.io default-kne-trigger
NAME                  READY   REASON        HOSTNAME   AGE
default-kne-trigger   False   TopicFailed              21h
❯ kubectl -n test-ns5 get brokers
NAME      READY   REASON      URL   AGE
default   False   NoAddress         24h

The knative-eventing-kafka-channel-controller

❯ kubectl -n knative-eventing logs -lapp.kubernetes.io/instance=knative-eventing-kafka -c channel-controller
...
"msg":"Failed To Create Topic (Results)","Controller":"kafka-channel-controller","Topic":"test-ns5.default-kne-trigger","error":"error code: 403, Details: SubCode=40300. The maximum number of resources of type EventHub has been reached or exceeded. Actual: 10, Max allowed: 10. For more information visit https://aka.ms/sbResourceMgrExceptions (Service Bus) or https://aka.ms/eventhubsarmexceptions (Event Hubs).

The application HTTP Source also won't be ready

❯ kubectl -n kyma-integration get httpsources test-app-7
NAME         READY   REASON
test-app-7   False   SinkNotFound
marcobebway commented 4 years ago

LGTM.

nachtmaar commented 4 years ago

Kyma behavior when reaching the quota limit of Azure Event hubs is documented.

@montaro The AC says the behaviour should be documented. I think it makes sense to have it in the public eventing documentation instead of only in a ticket. What do you think? cc: @anishj0shi