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

Add JDK11 and JDK(next) details for build pipelines #2414

Merged
merged 1 commit into from
Jul 20, 2018

Conversation

jdekonin
Copy link
Contributor

[skip ci]

Signed-off-by: Joe deKoning [email protected]

@jdekonin
Copy link
Contributor Author

jenkins test sanity all jdk11

@jdekonin
Copy link
Contributor Author

jenkins test extended all jdk11

@jdekonin
Copy link
Contributor Author

jdekonin commented Jul 19, 2018

Triggers work as expected. There appears to be VM and test material compile issues blocking passing runs.

This file needs to be refactored, but I don't want that to block the need to get JDK11 PR builds into place.

@AdamBrousseau please review.

@jdekonin jdekonin changed the title WIP: Add JDK11 and JDK(next) details for build pipelines Add JDK11 and JDK(next) details for build pipelines Jul 19, 2018
@@ -52,21 +58,29 @@ linux_ppc-64_cmprssptrs_le:
8: '/usr/lib/jvm/java-7-openjdk-ppc64el'
9: '/usr/lib/jvm/adoptojdk-java-ppc64le-80'
10: '/usr/lib/jvm/adoptojdk-java-ppc64le-90'
Copy link
Contributor

@AdamBrousseau AdamBrousseau Jul 20, 2018

Choose a reason for hiding this comment

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

is it a good time to remove the platform shorts from the 8,9,10 while we're here? Same for 390

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It would have been when I was adding jdk10 to the machines. I'll likely be updating jdk10 with something more current in the next month. I'll try remember to do that then.

@llxia llxia merged commit ff1126d into eclipse-openj9:master Jul 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants