-
Notifications
You must be signed in to change notification settings - Fork 979
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
Adding SASL_SSL dependencies for the rdkafka plugin #395
Conversation
Let's merge same 4 commits into one. |
59bf5ee
to
f0212ed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome :) I just can't figure out the python part, but everything else makes sense :) congrats!
0b1f157
to
c58f818
Compare
I think this PR is ready to be merge; @naumanrana 👍 Nice Job |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
Note that the installation of krb5-user is interactive which is why we add export DEBIAN_FRONTEND=noninteractive as per: https://askubuntu.com/questions/1017999/install-kerberos-client-without-interactive-session A post test is then done to make sure that rdkafka is linked against the sasl dependency properly Signed-off-by: naumanrana <[email protected]>
Thanks. New images will be published soon;. |
Thanks :) eager to use it :). BTW, I was looking at the issue #293 I would like to get your thoughts and know if we could have done something better. |
For SASL_SSL with GSSAPI to work with the rdkafka plugin, needed to install a bunch of kerberos related libraries and upgrade the fluent-plugin-kafka to the latest version as rdkafka was not available in 0.7.9.