Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8253 )
Change subject: mini-kdc: two workarounds to help Java tests pass in more environments ...................................................................... mini-kdc: two workarounds to help Java tests pass in more environments Environments with an older krb5 must enable the krb5_get_host_realm workaround. This was previously handled by OverrideKrb5Environment (gtests) and the Java MiniKdc, but now that the Java tests use the C++ MiniKdc, they stopped receiving the workaround. To fix, I moved the workaround from OverrideKrb5Environment to the C++ MiniKdc's GetEnvVars function. This ensures it gets enabled by any MiniKDC user, be it a standalone gtest or a full minicluster. Separately, not all JVMs include "unlimited strength crypto". The old Java MiniKdc explicitly disabled aes256 to accommodate these JVMs, but the C++ MiniKdc does not. Now it does. I tested this patch by running all C++ and Java unit tests on a machine that had libkrb5 1.10 and no "unlimited strength crypto" in its JVM. Change-Id: I6775d159c43e13213fb8f83b80a46b6c0818f468 Reviewed-on: http://gerrit.cloudera.org:8080/8253 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/security/test/mini_kdc.cc M src/kudu/util/test_util.cc 2 files changed, 13 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8253 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6775d159c43e13213fb8f83b80a46b6c0818f468 Gerrit-Change-Number: 8253 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
