Skip to content

KAFKA-21073: Enable SASL for the native image docker examples and sanity test - #23437

Open
muralibasani wants to merge 1 commit into
apache:trunkfrom
muralibasani:KAFKA-21073
Open

muralibasani wants to merge 1 commit into
apache:trunkfrom
muralibasani:KAFKA-21073

Conversation

@muralibasani

@muralibasani muralibasani commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Ref : https://issues.apache.org/jira/browse/KAFKA-21073

  • Remove the three stale "SASL not supported on native" notes from docker/examples/README.md and add the apache/kafka-native run command to each SASL example, for parity with the other examples.
  • Remove the MODE == "jvm" guard in docker_sanity_test.py so the SASL flow runs for the native image as well as the JVM image.

Reviewers: Chia-Ping Tsai chia7712@gmail.com

@github-actions github-actions Bot added triage PRs from the community docker Official Docker image small Small PRs labels Sep 11, 2026
@muralibasani

Copy link
Copy Markdown
Contributor Author

Tests locally

% python3 -m pip install -r docker/requirements.txt

% python3 docker/docker_build_test.py kafka-native-sanity -tag test -type native \
    --kafka-archive "$(pwd)/core/build/distributions/kafka_2.13-4.5.0-SNAPSHOT.tgz" -t

Running SASL Flow Tests
Created topic test-topic-sasl.
The consumer rebalance protocol (KIP-848) is production-ready! Set group.protocol=consumer to try it out. See https://kafka.apache.org/documentation/#consumer_rebalance_protocol
Processed a total of 1 messages

print(constants.FILE_INPUT_ERROR_PREFIX, str(e))
total_errors.append(str(e))
# SASL is not supported on native image due to missing reflection config (KAFKA-19584)
if self.MODE == "jvm":

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.

It seems the MODE is useless :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah yes, removed it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you for the review.

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

Labels

ci-approved docker Official Docker image small Small PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants