Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make all AdminClient timeouts configurable #498

Merged
merged 7 commits into from
Jan 2, 2025

Conversation

ThomasCAI-mlv
Copy link
Collaborator

No description provided.

@ThomasCAI-mlv ThomasCAI-mlv added the enhancement This issue or pull request improves a feature label Dec 20, 2024
@ThomasCAI-mlv ThomasCAI-mlv self-assigned this Dec 20, 2024
@ThomasCAI-mlv ThomasCAI-mlv linked an issue Dec 20, 2024 that may be closed by this pull request
@loicgreffier
Copy link
Collaborator

@ThomasCAI-mlv Updating this PR to make all the AdminClient timeouts configurable like so:

ns4kafka:
  managed-clusters:
    local:
      timeout:
        acl:
          create: 15000
          delete: 15000
          describe: 15000
        topic:
          alter-configs: 10000
          create: 10000
          describe-configs: 30000
          delete: 30000
          list: 30000
        user:
          alter-client-quotas: 10000
          alter-scram-credentials: 10000
          describe-quotas: 10000

@loicgreffier loicgreffier changed the title Make ACLs synchronisation timeout configurable Make all AdminClient timeout configurable Jan 2, 2025
@loicgreffier loicgreffier changed the title Make all AdminClient timeout configurable Make all AdminClient timeouts configurable Jan 2, 2025
@loicgreffier loicgreffier merged commit 0822de2 into master Jan 2, 2025
4 checks passed
@loicgreffier loicgreffier deleted the fix/acls-sync-timeout-parameter branch January 2, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue or pull request improves a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configurable timeout during ACL synchro for describeAcls call
2 participants