-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
STF test times out when daylight savings time changes #106
Comments
@Mesbah-Alam can you please take a look. |
The tests timed out, collected some diagnostics, and then finished afterwards. It seems the timeout occurred about 15 seconds after the tests started. |
Another one here with a 4 min timeout and with a less than 3 min timeout More: I get it, I believe this was caused by the daylight savings time change that occurred. What is the code used to determine if a timeout occurs? |
A human readable timeout value (e.g. Then that value gets parsed by STF here: https://github.com/AdoptOpenJDK/stf/blob/master/stf.core/src/stf.core/net/adoptopenjdk/stf/util/TimeParser.java#L40 It is possible for stf to mis-calculate the time for tests that start before the daylight saving time change takes place and are long enough to continue on after the change happens. |
Right. I'm wondering if the code can be improved so this problem can't occur. Otherwise this problem will occur every year until we get rid of daylight savings time changes. There are some java APIs which aren't affected by clock changes. |
I will look into it. Thanks for discovering the issue. FYI @lumpfish |
https://ci.eclipse.org/openj9/job/Test_openjdk8_j9_special.system_s390x_linux_Nightly/635
DaaLoadTest_all_special_27
variation: Mode612-OSRG
JVM_OPTIONS: -Xcompressedrefs -Xcompressedrefs -Xgcpolicy:gencon -Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
1.DLT.stdout shows both threads completed testing.
However a timeout occurred.
Looking at the javacore's collected, tests are still running. There is movement between the first javacore and the second.
The text was updated successfully, but these errors were encountered: