Skip to content
New issue

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

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

Already on GitHub? # to your account

Enable Zookeeper authentication #33

Closed

Conversation

rmarting
Copy link
Collaborator

This PR is an approach to implement the issue #32.

🚨 This PR is created on base of the content of the PR #28, so it is needed first to review the previous one and merge it. This is because the authentication mechanism must be aligned with the ZK and BK deployment topology.

The validation of both PR will cover one of the most standard deployment topology of a Kafka cluster.

@rmarting
Copy link
Collaborator Author

@rpelisse What do you think? This PR is aligned with the #28 one. Waiting for your feedback and comments.

@rmarting
Copy link
Collaborator Author

rmarting commented Jul 4, 2023

@rpelisse Merge done from the main branch. PR ready for your review!

@rpelisse
Copy link
Collaborator

rpelisse commented Jul 4, 2023

Superseeded by #42

@rpelisse rpelisse closed this Jul 4, 2023
@rmarting rmarting deleted the feat-zk-authentication branch July 18, 2023 08:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants