Skip to content

KAFKA-15513: Test SCRAM authentication for KRaft controllers - #23039

Open
lh0156 wants to merge 1 commit into
apache:trunkfrom
lh0156:agent/KAFKA-15513-controller-scram
Open

KAFKA-15513: Test SCRAM authentication for KRaft controllers#23039
lh0156 wants to merge 1 commit into
apache:trunkfrom
lh0156:agent/KAFKA-15513-controller-scram

Conversation

@lh0156

@lh0156 lh0156 commented Aug 2, 2026

Copy link
Copy Markdown

Summary

Add a focused system-test path for SCRAM authentication on isolated KRaft controller quorums. The test covers both SCRAM-SHA-256 and SCRAM-SHA-512.

The Kafka test service now bootstraps the controller quorum with the broker SCRAM credentials using kafka-storage.sh --add-scram, and the KafkaServer JAAS entry is selected from all enabled SASL mechanisms, including mechanisms used only by KRaft controller channels.

Motivation

KAFKA-15513 is not covered by the system-test harness: controller-quorum SCRAM was explicitly skipped, and formatted controller metadata did not contain the initial credentials needed for authentication.

Testing

  • ./gradlew :core:test --tests kafka.tools.StorageToolTest.testBootstrapScramRecords --no-daemon --console=plain
  • ./gradlew checkstyleMain checkstyleTest spotlessCheck --no-daemon --console=plain
  • Python compilation and targeted SCRAM bootstrap assertions
  • git diff --check
  • The full ducktape test is not run locally because the ducktape/worker environment is unavailable; CI should execute TestVerifiableProducer.test_isolated_kraft_controller_scram for both mechanisms.

Jira

https://issues.apache.org/jira/browse/KAFKA-15513

Add an isolated KRaft controller system test for SCRAM-SHA-256 and SCRAM-SHA-512. Bootstrap the controller quorum with the required credentials and render the broker JAAS SCRAM login for all enabled mechanisms.

Generated-by: OpenAI Codex (GPT-5)
@github-actions github-actions Bot added triage PRs from the community tests Test fixes (including flaky tests) small Small PRs labels Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

small Small PRs tests Test fixes (including flaky tests) triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant