Skip to content

Commit

Permalink
Mute kerberos tests for jdk 8u262 (#60353)
Browse files Browse the repository at this point in the history
The JDK bug (https://bugs.openjdk.java.net/browse/JDK-8246193) is fixed since b26.
The tests can be unmuted since we are already using b33. However the same bug is now
affecting jdk 8u262, which is the base for current Zulu jdk 8.48. This PR mute the tests
for this specific jdk version.

Relates: #56507
  • Loading branch information
ywangd committed Jul 29, 2020
1 parent 76bf8d2 commit 0299031
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,13 @@ public void setupRoleMapping() throws IOException {
assertOK(response);
}

// JDK 8u262 shipped with a NPE in Kerberos code, see https://github.com/elastic/elasticsearch/issues/56507
public void testSuppressedOnJDK8u262() {
assumeFalse("Cannot run on JDK 8u262", "1.8.0_262".equals(System.getProperty("java.version")));
}

public void testLoginByKeytab() throws IOException, PrivilegedActionException {
testSuppressedOnJDK8u262();
final String userPrincipalName = System.getProperty(TEST_USER_WITH_KEYTAB_KEY);
final String keytabPath = System.getProperty(TEST_USER_WITH_KEYTAB_PATH_KEY);
final boolean enabledDebugLogs = Boolean.parseBoolean(System.getProperty(ENABLE_KERBEROS_DEBUG_LOGS_KEY));
Expand All @@ -94,6 +100,7 @@ public void testLoginByKeytab() throws IOException, PrivilegedActionException {
}

public void testLoginByUsernamePassword() throws IOException, PrivilegedActionException {
testSuppressedOnJDK8u262();
final String userPrincipalName = System.getProperty(TEST_USER_WITH_PWD_KEY);
final String password = System.getProperty(TEST_USER_WITH_PWD_PASSWD_KEY);
final boolean enabledDebugLogs = Boolean.parseBoolean(System.getProperty(ENABLE_KERBEROS_DEBUG_LOGS_KEY));
Expand Down

0 comments on commit 0299031

Please sign in to comment.