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

[Test] Mute more certificate tests on JDK8 u292 #75994

Merged
merged 1 commit into from
Aug 3, 2021

Conversation

tvernum
Copy link
Contributor

@tvernum tvernum commented Aug 3, 2021

JDK 1.8.0_292 has a race condition that sometimes causes the
PBEWithSHA1AndDESede algorithm to appear as unavailable.

This change mutes additional tests on this JDK build

Relates: #75952, #75571, #75718

JDK 1.8.0_292 has a race condition that sometimes causes the
PBEWithSHA1AndDESede algorithm to appear as unavailable.

This change mutes additional tests on this JDK build

Relates: elastic#75952, elastic#75571, elastic#75718
@tvernum tvernum added :Security/TLS SSL/TLS, Certificates >test-mute Use for PR that only mute tests v7.15.0 labels Aug 3, 2021
@tvernum tvernum requested a review from ywangd August 3, 2021 01:40
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Aug 3, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -419,6 +418,9 @@ public void testGeneratingSignedPemCertificates() throws Exception {
}

public void testHandleLongPasswords() throws Exception {
assumeFalse("JDK bug JDK-8266279, https://github.com/elastic/elasticsearch/issues/75571",
Copy link
Member

Choose a reason for hiding this comment

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

Should the linked issue be #75952? Or are you using a single issue to track them all? Might be worthwhile to close all others if this is the case. But since the mute is specific to a particular version, I think we can just close the issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it's better to link them all to a single issue. Perhaps we should go back and change the description of the issue.

Tracking them in multiple issues makes it harder to handle once the next JDK build comes out and they all start failing (because the bug wasn't fixed) or we can remove these checks (if it is fixed).

@tvernum tvernum merged commit 6bc9972 into elastic:7.x Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Security/TLS SSL/TLS, Certificates Team:Security Meta label for security team >test-mute Use for PR that only mute tests v7.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants