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

[Enhancement]: use official kafka image #8398

Closed
rajadilipkolli opened this issue Feb 28, 2024 · 8 comments · Fixed by #8416
Closed

[Enhancement]: use official kafka image #8398

rajadilipkolli opened this issue Feb 28, 2024 · 8 comments · Fixed by #8416

Comments

@rajadilipkolli
Copy link

Module

Kafka

Proposal

From 3.7.0 version of kafka official image is being released which can be found at https://hub.docker.com/r/apache/kafka , Can we start supporting instead of using confluent.

I tried to use official image using .asCompitableSubstitue of confluent image but it is will not working.

@eddumelendez
Copy link
Member

Thanks for raising the issue! I would like to know if Kraft mode would be enough for you or are you expecting zookeeper support?

@rajadilipkolli
Copy link
Author

Hi @eddumelendez , as they started official image only from 3.7.0 where zookeeper is deprecated. I think only kraft mode is enough.

@Mitchell-Eaton
Copy link

Any updates on this? Our team would benefit from it. Thanks!

@rajadilipkolli
Copy link
Author

Hi @eddumelendez , from which version this is available to use ?

@eddumelendez
Copy link
Member

Hi, Testcontainers 1.19.8 contains org.testcontainers.kafka.KafkaContainer

@rajadilipkolli
Copy link
Author

Hi @eddumelendez ,

I am receiving below error when I replace confluent with apache/kafka one. Am I missing something?

org.springframework.boot.autoconfigure.service.connection.ConnectionDetailsNotFoundException: No ConnectionDetails found for source '@ServiceConnection source for Bean 'kafkaContainer' defined in com.example.service.TestServiceApplication'. You may need to add a 'name' to your @ServiceConnection annotation

@eddumelendez
Copy link
Member

It's a new implementation and spring boot doesn't support it yet. There is a open PR spring-projects/spring-boot#40695

@rajadilipkolli
Copy link
Author

Thank you for the swift response @eddumelendez

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants