-
Notifications
You must be signed in to change notification settings - Fork 721
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
Cannot run StringJoinerTest.java in jdknext grinder #11048
Comments
@llxia fyi |
Just for the record, I rerun 1133 (jdk16) with machine cent7-ppcle-5 (previously passed machine with jdk15). The Grinder also failed: https://ci.eclipse.org/openj9/view/Test/job/Grinder/1135/ |
Used Grinder/11672 and ran the test on rhel7lert4 (internally). The test failed with
Currently, we are using https://ci.adoptopenjdk.net/job/jtreg/lastSuccessfulBuild/artifact/jtreg-4.2.0-tip.tar.gz. I also tried https://ci.adoptopenjdk.net/job/jtreg/lastSuccessfulBuild/artifact/jtreg-5.1-b01.tar.gz. It does not seem to make any difference. (see https://ci.eclipse.org/openj9/view/Test/job/Grinder/1139/) @sophia-guo Any idea of how jtreg checks |
I think it's due to vm.bits is not added as the requires properties in jdk components. Will check if it's correct. |
openjdk use VMProps.java set up Properties used by @requires expressions, which setup vm.bits. openj9 use OpenJ9PropsExt.java setup Properties , which hasn't added vm.bits property. |
Take 15 for example
https://github.com/sophia-guo/openj9-openjdk-jdk/blob/openj9/closed/test/jtreg-ext/requires/OpenJ9PropsExt.java need to be updated and back port to 11. |
The fix is delivered in ibmruntimes/openj9-openjdk-jdk#246 |
jdk15 grinder https://ci.eclipse.org/openj9/view/Test/job/Grinder/1133 is working.
jdknext grinder https://ci.eclipse.org/openj9/view/Test/job/Grinder/1132 shows
no tests selected
.I did notice the test requirements are different in jdknext
@requires vm.bits == "64" & os.maxMemory > 4G
vs@requires os.maxMemory > 4G
for jdk15.The text was updated successfully, but these errors were encountered: