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

Initial Windows ARM64 problem listing #4159

Merged

Conversation

joe-braley
Copy link
Contributor

Hey team!

Related GH issue: #4155. Please let me know if this issue requires splitting up.

java/foreign/* tests might impact the temurin builds as well so included those in the main problem lists.  I also believe that jdk/sun/tools/jhsdb/JShellHeapDumpTest.java might impact that build as well since there have been similar issues with that test on windows in the past (see: https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8230731). But as always, I could be mistaken.

Please let me know if there are any questions or concerns with this PR!

Thanks!

CC @karianna / @gdams / @ShelleyLambert for visibility. 

@sophia-guo
Copy link
Contributor

@joe-braley Will Microsoft run openjdk tests with with jdk8 or other versions? If yes the corresponding vendor ProblemList file will also need to be added even if it might be empty file without tests excluded.

@joe-braley
Copy link
Contributor Author

joe-braley commented Nov 28, 2022

@sophia-guo We currently run openjdk tests against 11 and 17. I believe the next version we will support will be 21 which comes out September 2023 if I am not mistaken. I can add the empty problem lists if that is still a requirement! Will add the additional problem lists.

Copy link
Contributor

@sophia-guo sophia-guo left a comment

Choose a reason for hiding this comment

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

LGTM

@smlambert smlambert merged commit 1061a87 into adoptium:master Nov 29, 2022
@joe-braley joe-braley deleted the joe/add-msft-winarm-problemlistings branch November 29, 2022 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants