Skip to content

Add KIP-546 client quota Admin APIs - #2321

Draft
Honglei (honglei) wants to merge 1 commit into
confluentinc:masterfrom
honglei:integration/kip-546-python
Draft

Add KIP-546 client quota Admin APIs#2321
Honglei (honglei) wants to merge 1 commit into
confluentinc:masterfrom
honglei:integration/kip-546-python

Conversation

@honglei

Copy link
Copy Markdown

What

Expose KIP-546 describe_client_quotas() and alter_client_quotas() through the Python AdminClient. This adds the public quota models, C-extension conversion/result handling, type stubs, validation, unit tests, and a Kafka-backed roundtrip test.

This is a stacked draft and depends on confluentinc/librdkafka#5560. It must not be merged until that C API is accepted and the bundled librdkafka revision is updated.

Checklist

  • Contains customer facing changes, including new Admin APIs
  • Adds unit and integration test coverage

References

Test & Review

Built the extension against the exact reviewed tree from librdkafka#5560, then ran:

  • black, isort, and flake8 on changed Python files
  • mypy: 119 source files, no issues
  • quota/Admin unit suite: 35 passed, 2 unrelated DNS timing tests deselected
  • Kafka 4.3.1 integration roundtrip: validate-only, set, describe, and remove

Open questions / Follow-ups

The PR remains draft until librdkafka#5560 lands and this repository updates its librdkafka dependency.

Expose DescribeClientQuotas and AlterClientQuotas through the Python AdminClient, with immutable quota entity keys, input validation, type stubs, unit coverage, and a Kafka-backed roundtrip test.
@confluent-cla-assistant

Copy link
Copy Markdown

🎉 All Contributor License Agreements have been signed. Ready to merge.
✅ honglei
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant