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

feat: Capability in dagger to consume from ACL enabled kafka clusters #195

Merged
merged 14 commits into from
Oct 17, 2022
Merged

feat: Capability in dagger to consume from ACL enabled kafka clusters #195

merged 14 commits into from
Oct 17, 2022

Conversation

mayankrai09
Copy link
Contributor

we are working on a centralized Kafka cluster that would contain data from other entities (Gojek/Gofin..etc). For compliance reasons, the cluster needs to be ACL enabled. Also, there is a need to run daggers that would consume from this ACL-enabled cluster.
We are planning to use
securtiy_protocol: SASL_PLAINTEXT
sasl_mechanism: SCRAM-SHA-512/SCRAM-SHA-256 [We took the decision to use SCRAM for the ease of adding new users]
Apart from these configs we would also need to have jaas-config in the consumer side containing the user credentials. We would be mounting the jaas-config to that particular flink cluster which would be consuming from ACL-enabled Kafka clusters.

version.txt Outdated Show resolved Hide resolved
@Meghajit
Copy link
Member

@prakharmathur82 @mayankrai09 This seems like an important but advanced feature, which might be tricky to configure for new users.

Hence I was thinking if we should include a separate page under the Advance section here in the documentation to help guide folks on what is Kerberos authentication in Dagger, which sources support it and how to enable it by setting which configuration variables.

Maybe titled Security, similar to how Flink does it : https://nightlies.apache.org/flink/flink-docs-release-1.13/docs/deployment/security/security-kerberos/

cc: @ravisuhag

Copy link
Member

@Meghajit Meghajit left a comment

Choose a reason for hiding this comment

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

Have added some comments on the PR

@Meghajit Meghajit merged commit 8f32ecc into raystack:main Oct 17, 2022
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.

4 participants