-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add ConfluentKafkaContainer #9139
Conversation
## Benefits | ||
|
||
* Running a single node Kafka installation with just one line of code | ||
* No need to manage external Zookeeper installation, required by Kafka. But see [below](#zookeeper) | ||
|
||
## Example | ||
|
||
### Using org.testcontainers.containers.KafkaContainer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we replace this documentation sections with org.testcontainers.kafka.KafkaContainer
as well? I think accordingly, we can even remove the docs for the deprecated version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For those interested in kafka and zookeeper, this could be helpful. Otherwise, it would look like it is not supported.
!!! note | ||
Only available for `org.testcontainers.containers.KafkaContainer` | ||
|
||
KRaft mode was declared production ready in 3.3.1 (confluentinc/cp-kafka:7.3.x) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Related to above, I think we then can remove docs for the deprecated class.
Co-authored-by: Kevin Wittek <kiview@users.noreply.github.com>
org.testcontainers.containers.KafkaContainer
org.testcontainers.kafka.ConfluentKafkaContainer
which works withconfluentinc/cp-kafka
images with version7.4.0
or later.KafkaHelper
container common env vars, command, wait strategy to be shared withorg.testcontainers.kafka.KafkaContainer