Skip to content

chore(fvt): refactor docker-compose and support KRaft#3323

Merged
dnwe merged 2 commits intomainfrom
dnwe/better-fvt
Oct 1, 2025
Merged

chore(fvt): refactor docker-compose and support KRaft#3323
dnwe merged 2 commits intomainfrom
dnwe/better-fvt

Conversation

@dnwe
Copy link
Copy Markdown
Collaborator

@dnwe dnwe commented Oct 1, 2025

  • make use of docker compose 'fragments' (yaml anchors) to reduce a lot of duplication in the docker-compose
  • remove the pinned commit we were using to test Kafka 4.0 with ZooKeeper support and start testing against the release with KRaft enabled
  • use the entrypoint.sh to remove the inappropriate broker properties if in zookeeper mode or kraft mode respectively
  • don't start the zookeeper containers unless --profile zookeeper is passed, this saves on the resource at the cost of not necessarily being obvious to a casual user. It's a shame we can't selectively start them based on KAFKA_VERSION

@dnwe dnwe added the chore label Oct 1, 2025
- make use of docker compose 'fragments' (yaml anchors) to reduce a lot
  of duplication in the docker-compose
- remove the pinned commit we were using to test Kafka 4.0 with
  ZooKeeper support and start testing against the release with KRaft
  enabled
- use the entrypoint.sh to remove the inappropriate broker properties if in
  zookeeper mode or kraft mode respectively
- don't start the zookeeper containers unless `--profile zookeeper` is
  passed, this saves on the resource at the cost of not necessarily
  being obvious to a casual user. It's a shame we can't selectively
  start them based on KAFKA_VERSION

Signed-off-by: Dominic Evans <[email protected]>
@dnwe dnwe force-pushed the dnwe/better-fvt branch from fb2bb60 to ca9f934 Compare October 1, 2025 12:40
When running Kafka in KRaft mode there seems to be additional latency
between adding a client quota and it being available on a
DescribeClientQuotas call. Add a few short retries in the test to
account for that lag.

Signed-off-by: Dominic Evans <[email protected]>
@dnwe dnwe merged commit dcea0ff into main Oct 1, 2025
20 of 21 checks passed
@dnwe dnwe deleted the dnwe/better-fvt branch October 1, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants