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

Scope clients to new hosts #42

Merged
merged 1 commit into from
Dec 7, 2024
Merged

Scope clients to new hosts #42

merged 1 commit into from
Dec 7, 2024

Conversation

andrewstucki
Copy link
Contributor

This allows for creating a "scoped" client that talks only to the given broker, but with the same TLS and auth settings as the previous client. It adds methods for scoping a client to a broker by host URL and for scoping via Broker ID.

Copy link
Contributor

@RafalKorepta RafalKorepta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this AdminAPI client have integration tests? Maybe simple test-container with single Redpanda process that could talk with that client.

Never the less I would like to use that feature 👍

@bojand bojand merged commit dd5f6c3 into main Dec 7, 2024
2 checks passed
@bojand bojand deleted the scoped-clients branch December 7, 2024 01:04
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.

3 participants