-
Notifications
You must be signed in to change notification settings - Fork 722
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
JDK11 Windows jdk_security java/security/SecureRandom/ApiTest.java & EnoughSeedTest.java timed out on some machines #16734
Comments
https://openj9-jenkins.osuosl.org/job/Test_openjdk11_j9_sanity.openjdk_x86-64_windows_Nightly/560/ - win2019-x64-1a |
JDK11 x86-64_windows 0.40.0 rc1(
|
pshipton
added a commit
to pshipton/openjdk-tests
that referenced
this issue
Aug 15, 2023
These tests fail often enough on Windows to exlude them. java/security/SecureRandom/ApiTest.java java/security/SecureRandom/EnoughSeedTest.java Issue eclipse-openj9/openj9#16734 Signed-off-by: Peter Shipton <[email protected]>
Created adoptium/aqa-tests#4707 to exclude the tests on Windows. |
llxia
pushed a commit
to adoptium/aqa-tests
that referenced
this issue
Aug 16, 2023
…ws (#4707) These tests fail often enough on Windows to exlude them. java/security/SecureRandom/ApiTest.java java/security/SecureRandom/EnoughSeedTest.java Issue eclipse-openj9/openj9#16734 Signed-off-by: Peter Shipton <[email protected]>
pshipton
added a commit
to pshipton/openjdk-tests
that referenced
this issue
Aug 17, 2023
Accidently excluded them only for 32-bit in adoptium#4707 Issue eclipse-openj9/openj9#16734 Signed-off-by: Peter Shipton <[email protected]>
pshipton
added a commit
to pshipton/openjdk-tests
that referenced
this issue
Aug 17, 2023
Accidentally excluded them only for 32-bit in adoptium#4707 Issue eclipse-openj9/openj9#16734 Signed-off-by: Peter Shipton <[email protected]>
smlambert
pushed a commit
to adoptium/aqa-tests
that referenced
this issue
Aug 17, 2023
#4710) Accidentally excluded them only for 32-bit in #4707 Issue eclipse-openj9/openj9#16734 Signed-off-by: Peter Shipton <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Failure link
From an internal build(
win19x86-rt2-1
):Rerun in Grinder - Change TARGET to run only the failed test targets.
Optional info
Failure output (captured from console output)
50x grinder - 11/50 failed
-Djdk.nativeCrypto=false
doesn't help.Adoptium JDK11 build has similar errors:
https://openj9-jenkins.osuosl.org/job/Grinder_iteration_2/183 - win2019-x64-1
https://openj9-jenkins.osuosl.org/job/Grinder_iteration_3/153 - win2019-x64-2
https://openj9-jenkins.osuosl.org/job/Grinder_iteration_5/67 - win2012x64-openj9-1
From an internal investigation link
The specific test(s) timed out. There are two tests but they are both running the ThreadedSeedGenerator.
From the output you can see it's running the following code, the test is waiting on this.
In some of my grinders I collected diagnostics (javacore, core, etc.), and since my PR was merged this morning all of these runs will collect diagnostics. It's burning a lot of system CPU time (16+ min) running the ThreadedSeedGenerator loop. I suspected a JIT issue but the timeout also occurs with Adoptium. We could try increasing the timeout for the test, but I'm not sure it's worth it considering it already takes so long. For now we can just treat it as a known issue. It's possible the ThreadedSeedGenerator code is somewhat bogus, it looks to me like the loop counter < 64000 condition could result in running for a long time if the latch value, which loops around, ends up being quite small.
FYI @pshipton
The text was updated successfully, but these errors were encountered: