I'm not sure if it's not better to keep the KAFKA_BROKER_ID closed to the instance declaration as it needs to be unique in a cluster and can't be shared between the instances.
But otherwise, thank, it's simpler like that.

This revision is now accepted and ready to land.Aug 8 2022, 1:45 PM

Rebase

Harbormaster completed remote builds in B30709: Diff 29641.Aug 9 2022, 10:03 AM

rebase

Harbormaster completed remote builds in B30714: Diff 29646.Aug 9 2022, 10:12 AM

Closed by commit rDENV8d05ddb97216: docker: move all 'static' kafka config entries in env/kafka.env (authored by douardda). · Explain WhyAug 9 2022, 11:12 AM

This revision was automatically updated to reflect the committed changes.

douardda added a commit: rDENV8d05ddb97216: docker: move all 'static' kafka config entries in env/kafka.env.

Revision Contents
Changeset List

Path

Size

docker/

docker-compose.yml

6 lines

env/

kafka.env

10 lines

Diff 29654

View Options

docker/docker-compose.yml

View Options

docker: move all 'static' kafka config entries in env/kafka.envClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 29654

docker/docker-compose.yml

docker/env/kafka.env

docker: move all 'static' kafka config entries in env/kafka.env
ClosedPublic
Actions

Revision Contents
Changeset List