Skip to content

KAFKA-21269: Add tests for describing broker configs with raw string values - #23859

Open
lucliu1108 wants to merge 3 commits into
apache:trunkfrom
lucliu1108:lucliu/KAFKA-21269-describe-broker-config-raw-string-values-test
Open

lucliu1108 wants to merge 3 commits into
apache:trunkfrom
lucliu1108:lucliu/KAFKA-21269-describe-broker-config-raw-string-values-test

Conversation

@lucliu1108

@lucliu1108 lucliu1108 commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Cluster-wide broker configs and listener-prefixed broker configs reach
createBrokerConfigEntry as raw strings instead of parsed values. For
LIST configs, passing them to ConfigDef.convertToString would throw
ClassCastException, so a guard returns them unchanged. No test covered
this guard yet.

This PR adds two tests to DescribeConfigsTest:

  1. testDescribeConfigsClusterWideListConfig: sets
    log.cleanup.policy=compact,delete as a cluster-wide dynamic config and
    describes it.
  2. testDescribeConfigsPrefixedListenerListConfig: sets
    listener.name.external.ssl.enabled.protocols as a static broker config
    and describes it.

@github-actions github-actions Bot added triage PRs from the community tests Test fixes (including flaky tests) clients small Small PRs labels Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved clients 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.

2 participants