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

KAFKA-5173: Add logging to dump JaasContext before writing it to file #2981

Closed
wants to merge 1 commit into from

Conversation

baluchicken
Copy link
Contributor

@rajinisivaram plz review.

@asfbot
Copy link

asfbot commented May 5, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/3538/
Test FAILed (JDK 8 and Scala 2.12).

@asfbot
Copy link

asfbot commented May 5, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/3547/
Test FAILed (JDK 8 and Scala 2.11).

@asfbot
Copy link

asfbot commented May 5, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/3541/
Test FAILed (JDK 7 and Scala 2.10).

@rajinisivaram
Copy link
Contributor

@baluchicken Thank you for the PR. There was a failure with the same symptoms with this PR (https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/3547/testReport/kafka.api/SaslGssapiSslEndToEndAuthorizationTest/testTwoConsumersWithDifferentSaslCredentials/). Unfortunately, only warning level logs are available, so the extra info level log didn't help. Can you see if you can catch the server startup exception and log the jaasContext that was written and then rethrow the exception? If you can't get to the jaasContext easily, you could just read the file based on the system property and display that (at error level in the catch block).

@rajinisivaram
Copy link
Contributor

@baluchicken I have raised another PR to log the contents of the file when there is a failure.

@baluchicken
Copy link
Contributor Author

Thanks, @rajinisivaram

@baluchicken baluchicken closed this May 8, 2017
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