Skip to content

KAFKA-20982: Enable SASL_PLAINTEXT listener on all brokers in docker sanity test fixtures - #23263

Merged
chia7712 merged 3 commits into
apache:trunkfrom
mingyen066:KAFKA-20982-sasl-listeners-all-brokers
Aug 26, 2026
Merged

KAFKA-20982: Enable SASL_PLAINTEXT listener on all brokers in docker sanity test fixtures#23263
chia7712 merged 3 commits into
apache:trunkfrom
mingyen066:KAFKA-20982-sasl-listeners-all-brokers

Conversation

@mingyen066

@mingyen066 mingyen066 commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Only broker1 had a SASL_PLAINTEXT listener, but the test topic (1
partition, RF=1) and the group coordinator can land on broker2/broker3,
causing the client to time out (reported in
#22023 (comment)). Add
the listener to all brokers in both isolated and combined mode,
following the existing SSL pattern.

Reviewers: Chia-Ping Tsai chia7712@gmail.com

@github-actions github-actions Bot added triage PRs from the community tests Test fixes (including flaky tests) docker Official Docker image small Small PRs labels Aug 25, 2026
@mingyen066

Copy link
Copy Markdown
Collaborator Author

Verified locally by running the docker sanity tests against apache/kafka:4.3.1:

python docker_build_test.py apache/kafka --image-tag=4.3.1 --image-type=jvm --test --kafka-archive=/tmp/kafka_2.13-4.3.1.tgz

Both combined and isolated modes passed, including the SASL flow:

Created topic test-topic-sasl.
Processed a total of 1 messages
...
All tests passed successfully.

@chia7712

Copy link
Copy Markdown
Member

python docker_build_test.py apache/kafka --image-tag=4.3.1 --image-type=jvm --test --kafka-archive=/tmp/kafka_2.13-4.3.1.tgz

This won't trigger the build flow, so it luckily skips the error mentioned in #22054 (comment) :)

Anyway, would you mind fixing it as well?

@mingyen066

Copy link
Copy Markdown
Collaborator Author

@chia7712 Thanks for catching this! Fixed and verified the full build + test flow:

❯ python docker_build_test.py kafka/test --image-tag=local-verify --image-type=jvm --kafka-archive=/tmp/kafka_2.13-4.3.1.tgz
[+] Building 29.0s (18/18) FINISHED                                     docker:orbstack
 => [internal] load build definition from Dockerfile                               0.0s
...
Time Elapsed: 0:05:58.472633
All tests passed successfully.

@chia7712 chia7712 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chia7712
chia7712 merged commit 8eb07f3 into apache:trunk Aug 26, 2026
22 checks passed
chia7712 pushed a commit that referenced this pull request Aug 26, 2026
…anity test fixtures (#23263)

Only broker1 had a SASL_PLAINTEXT listener, but the test topic (1
partition, RF=1) and the group coordinator can land on broker2/broker3,
causing the client to time out (reported in
#22023 (comment)). Add
the listener to all brokers in both isolated and combined mode,
following the existing SSL pattern.

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
@chia7712

Copy link
Copy Markdown
Member

backport to 4.4 branch

@github-actions github-actions Bot removed the triage PRs from the community label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Official Docker image small Small PRs tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants