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

Openjdk 11+ build need also build test-image #248

Closed
sophia-guo opened this issue Mar 6, 2018 · 18 comments · Fixed by #1251
Closed

Openjdk 11+ build need also build test-image #248

sophia-guo opened this issue Mar 6, 2018 · 18 comments · Fixed by #1251
Assignees
Labels
enhancement Issues that enhance the code or documentation of the repo in any way

Comments

@sophia-guo
Copy link
Contributor

sophia-guo commented Mar 6, 2018

Some tests of test/jdk and test/hotspot need jdk test native library, which has to be generated at build time of the test jdk by invoking "make test-image"

It appears as though the native test libraries are currently not being built as part of the Java10 builds at AdoptOpenJDK, as we do not find libraries under images/test/jdk/jtreg/native for those builds.

https://download.java.net/openjdk/testresults/10/docs/howtoruntests.html

-nativepath:{location of the jdk test native library}: This has to be generated at build time of the test jdk by invoking "make test-image". These libraries are located in the build directory under "images/test/jdk/jtreg/native/"

@karianna karianna added the bug Issues that are problems in the code as reported by the community label Mar 7, 2018
@karianna
Copy link
Contributor

#263 should resolve this.

@sophia-guo
Copy link
Contributor Author

sophia-guo commented Apr 26, 2018

#263 build test-image, which is under same folder level of jdk or ${OPENJDK_REPO_TAG} - named test. However when doing archive only ${OPENJDK_REPO_TAG} is tar-ed. createOpenJDKTarArchive() need to update so when untared jdk folder createOpenJDKTarArchive() and test are expected.

@sophia-guo
Copy link
Contributor Author

Given it a second thought we probably need to tar the JDK and TestImage separately as TestImage should not be part of released sdk? The story would be like: SDK build create SDK and testImage , test build consume SDK and testImage, and then if test build is good SDK itself will be published (testImage is ignored as it should not be part of releases) ?

@karianna
Copy link
Contributor

@sophia-guo Is this needed for Java 11? If not then we'll close as we're no longer supporting Java 10.

@karianna karianna added the Waiting on OP Issues that are awaiting a response from the original author of the ticket label Jan 10, 2019
@sophia-guo
Copy link
Contributor Author

yes, this is also required for Java11. It is required for 10+.

@karianna karianna added this to the January 2019 milestone Jan 10, 2019
@karianna karianna added enhancement Issues that enhance the code or documentation of the repo in any way and removed Waiting on OP Issues that are awaiting a response from the original author of the ticket bug Issues that are problems in the code as reported by the community labels Jan 10, 2019
@karianna karianna modified the milestones: January 2019, February 2019 Feb 3, 2019
@smlambert smlambert changed the title Openjdk 10 build need also build test-image Openjdk 11+ build need also build test-image Feb 13, 2019
@navyxliu
Copy link

I am working on Corretto11 regression test. I am very interested in this issue and its resolution.

So far, I have identified 10 test failures due to 'Error. Use -nativepath to specify the location of native code'

java/nio/channels/FileChannel/directio/DirectIOTest.java
java/nio/channels/spi/SelectorProvider/inheritedChannel/InheritedChannelTest.java
compiler/aot/calls/fromAot/AotInvokeDynamic2NativeTest.java
compiler/aot/calls/fromAot/AotInvokeInterface2NativeTest.java
compiler/aot/calls/fromAot/AotInvokeSpecial2NativeTest.java
compiler/aot/calls/fromAot/AotInvokeStatic2NativeTest.java
compiler/aot/calls/fromAot/AotInvokeVirtual2NativeTest.java
compiler/aot/calls/fromNative/NativeInvokeSpecial2AotTest.java
compiler/aot/calls/fromNative/NativeInvokeStatic2AotTest.java
compiler/aot/calls/fromNative/NativeInvokeVirtual2AotTest.java

@jerboaa
Copy link
Contributor

jerboaa commented Feb 13, 2019

IMO this isn't entirely a openjdk-build issue only. What if somebody has a build with test libs already? I'd imagine the test framework should allow for a path to be set via some environment variable which is then passed to jtreg via -nativepath.

@smlambert
Copy link
Contributor

Agreed @jerboaa - as discussed in Slack, you can leverage JTREG_EXTRA_OPTIONS for the use case of having your own build, and a path to the native test libs.

This issue is more for us to be able to automate native testing in the AdoptOpenJDK pipelines.

jerboaa added a commit to jerboaa/temurin-build that referenced this issue Aug 27, 2019
jerboaa added a commit to jerboaa/temurin-build that referenced this issue Aug 27, 2019
@karianna karianna modified the milestones: August 2019, September 2019 Sep 2, 2019
jerboaa added a commit to jerboaa/temurin-build that referenced this issue Sep 2, 2019
jerboaa added a commit to jerboaa/temurin-build that referenced this issue Sep 3, 2019
@jerboaa
Copy link
Contributor

jerboaa commented Sep 3, 2019

@smlambert @sophia-guo: This job has the testimage artifact as well for early testing. Let me know if that'll work for you:
https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk11u/job/jdk11u-linux-x64-hotspot/133/

@sophia-guo
Copy link
Contributor Author

Thanks @jerboaa. I will try today or tomorrow and let you know.

jerboaa added a commit to jerboaa/temurin-build that referenced this issue Sep 3, 2019
@sophia-guo
Copy link
Contributor Author

@jerboaa the prtestbuid https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk11u/job/jdk11u-linux-x64-hotspot/133/ doesn't have any /*.tar.gz,/.tgz,**/.zip,**/*.jar artifacts. Is there any other way that I can download the test image you have built and archived? Ideally it should be available same as other build artifacts, e.t.c. jre|jdk iamge artifacts.

@sophia-guo
Copy link
Contributor Author

@jerboaa it's great! thanks.

@jerboaa
Copy link
Contributor

jerboaa commented Sep 4, 2019

@sophia-guo Glad to hear that. Just need to get the PR merged then ;-)

jerboaa added a commit to jerboaa/temurin-build that referenced this issue Sep 6, 2019
@sxa sxa unpinned this issue Nov 28, 2019
babsingh added a commit to babsingh/aqa-tests that referenced this issue Jan 12, 2022
- eclipse-openj9/openj9#14079 is a duplicate of adoptium/temurin-build#248.
- eclipse-openj9/openj9#14091 is a duplicate of adoptium#1297.
- eclipse-openj9/openj9#14095 is a duplicate of eclipse-openj9/openj9#11930.
- Individual sub-tests of StringBuilder/HugeCapacity are excluded.

Closes: eclipse-openj9/openj9#14079
Closes: eclipse-openj9/openj9#14091
Closes: eclipse-openj9/openj9#14095

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
babsingh added a commit to babsingh/aqa-tests that referenced this issue Jan 13, 2022
adoptium/temurin-build#248 has been closed. All tests disabled for
adoptium/temurin-build#248 have been re-enabled.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
babsingh added a commit to babsingh/aqa-tests that referenced this issue Jan 13, 2022
adoptium/temurin-build#248 has been closed. All tests disabled for
adoptium/temurin-build#248 have been re-enabled.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
babsingh added a commit to babsingh/aqa-tests that referenced this issue Jan 13, 2022
Fixed by ibmruntimes/openj9-openjdk-jdk18#4.
Also, adoptium/temurin-build#248 has been closed.
Now, exeCallerAccessTest/CallerAccessTest passes with OpenJ9 JDK18.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
LongyuZhang pushed a commit to adoptium/aqa-tests that referenced this issue Jan 13, 2022
* [JDK18] Remove duplicates and exclude sub-tests

- eclipse-openj9/openj9#14091 is a duplicate of #1297.
- eclipse-openj9/openj9#14095 is a duplicate of eclipse-openj9/openj9#11930.
- Individual sub-tests of StringBuilder/HugeCapacity are excluded.

Closes: eclipse-openj9/openj9#14091
Closes: eclipse-openj9/openj9#14095

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>

* [JDK18] Re-enable exeCallerAccessTest/CallerAccessTest

Fixed by ibmruntimes/openj9-openjdk-jdk18#4.
Also, adoptium/temurin-build#248 has been closed.
Now, exeCallerAccessTest/CallerAccessTest passes with OpenJ9 JDK18.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issues that enhance the code or documentation of the repo in any way
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants