Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(docs): add note that Basic tier in Event Hub does not support Kafka protocol #5018

Merged
merged 2 commits into from
Aug 10, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,13 @@

Seldon Core v2 can integrate with Azure Event Hub via Kafka protocol.

```{warning}
You will need at least `Standard` tier for your Event Hub Namespace as `Basic` tier does not support Kafka protocol.
```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💭 Having two warning blocks after one another is a bit noisy visually. A single block with two paragraphs would be easier to read.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are only semi related so I should leave them independent as they provide separate bits of informations.


```{warning}
Seldon Core v2 creates 2 Kafka topics for each pipeline and model plus one global topic for errors.
This means that total number of topics will be `2 x (#models + #pipelines) + 1` which will likely exceed the limit of `Basic` and `Standard` tier in Azure Event Hub.
This means that total number of topics will be `2 x (#models + #pipelines) + 1` which will likely exceed the limit of `Standard` tier in Azure Event Hub.
See quota information [here](https://learn.microsoft.com/en-us/azure/event-hubs/event-hubs-quotas#basic-vs-standard-vs-premium-vs-dedicated-tiers).
RafalSkolasinski marked this conversation as resolved.
Show resolved Hide resolved
```

Expand Down