To make Kafka accessible from IDE on port 9093

[kmozaid]

No description provided.

[jpechane]

@kmozaid Hi, thanks for the PR

• Is it ok to advertise endpoint that is not accessible for some of the clients?
• WOuld it be possible to introduce the same change into all tutorial composes?

[kmozaid]

@kmozaid Hi, thanks for the PR

• Is it ok to advertise endpoint that is not accessible for some of the clients?
• WOuld it be possible to introduce the same change into all tutorial composes?

HI @jpechane

• The endpoint EXTERNAL://localhost:9093 is advertised so that clients on docker host can access Kafka running in docker. If we specify this external endpoint then we also need to advertise endpoint for internal service like kafka connect (The endpoint CLIENT://kafka:9092 is for Kafka Connect Docker).
• Sure, I can update other docker-compose files.

[rk3rn3r]

Quick question: Is it valuable to map to a static port? Won't this conflict with multiple containers running concurrently? Would it be possible to use random ports and map the config in the entrypoint script?
Else, we should at least mind in the docs that this can conflict and running the same container multiple times isn't supported.
wdyt @jpechane ?

[OneCricketeer]

Won't this conflict with multiple containers running concurrently

Yes.

possible to use random ports and map the config in the entrypoint script

advertised.listeners needs to have a static value, albeit one could be dynamically generated, but then how would compose or the Dockerfile expose that random value without defining it statically in those configs?

[kmozaid]

@OneCricketeer and @jpechane Sorry for the very late update on this PR, please review once again, I have made the required changes.

[Naros]

❌ Developer Certificate of Origin (DCO) check failed.

Hi @kmozaid, please sign off all commits with:

git commit -s

If pull request commits are not signed off, the pull request cannot be merged. For more information about why this is required, please see our blog about contribution requirement changes.