You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "stasimus (via GitHub)" <gi...@apache.org> on 2026/08/22 08:57:19 UTC
[PR] MINOR: Do not advertise describe APIs on the controller listener [kafka]
stasimus opened a new pull request, #23237:
URL: https://github.com/apache/kafka/pull/23237
DescribeDelegationToken and DescribeUserScramCredentials declare the controller listener, so the controller advertises them in ApiVersions, but `ControllerApis.handle` has no case for either and clients get UNKNOWN_SERVER_ERROR for an API that was just advertised.
Dropped the controller listener so the advertised set matches what is implemented. Adding handlers instead would be new functionality. Happy to route this through a KIP if the ApiVersions change counts as a protocol change.
Both were introduced with their KRaft features (KAFKA-15219, KAFKA-14084), which wired up the mutating APIs but not the describes. Added two regression tests in `ControllerApisTest`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: Do not advertise describe APIs on the controller listener [kafka]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #23237:
URL: https://github.com/apache/kafka/pull/23237#issuecomment-5466440419
A label of 'needs-attention' was automatically added to this PR in order to raise the
attention of the committers. Once this issue has been triaged, the `triage` label
should be removed to prevent this automation from happening again.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: Do not advertise describe APIs on the controller listener [kafka]
Posted by "stasimus (via GitHub)" <gi...@apache.org>.
stasimus commented on PR #23237:
URL: https://github.com/apache/kafka/pull/23237#issuecomment-5379438913
changes to the wire protocol "generally require a KIP", and this does change what the controller returns in its ApiVersions response - I take it a bugfix rather than a protocol change, since the API never worked on the controller and no functioning client behaviour change
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org