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

Valhalla disables JDK22+ Jep454Tests temporarily #18584

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

JasonFengJ9
Copy link
Member

@JasonFengJ9 JasonFengJ9 commented Dec 8, 2023

Valhalla disables JDK22+ Jep454Tests temporarily

Valhalla extension repo is well behind OpenJDK head, and doesn't support some latest APIs. Disabled Jep454Tests temporarily until the valhalla project is updated w/ a recent OpenJDK tag.

related

FYI @hangshao0

Signed-off-by: Jason Feng [email protected]

Valhalla extension repo is well behind OpenJDK head, and doesn't support
some latest APIs. Disabled Jep454Tests temporarily until the valhalla
project is updated w/ a recent OpenJDK tag.

Signed-off-by: Jason Feng <[email protected]>
@JasonFengJ9 JasonFengJ9 added comp:test project:valhalla Used to track Project Valhalla related work labels Dec 8, 2023
@hangshao0
Copy link
Contributor

Jenkins test sanity winvalst jdknext

@hangshao0
Copy link
Contributor

The failures in the PR build are not related to this change.

Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia llxia merged commit 0c87adb into eclipse-openj9:master Dec 11, 2023
3 of 5 checks passed
@JasonFengJ9 JasonFengJ9 deleted the valdisabletests branch December 11, 2023 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:test project:valhalla Used to track Project Valhalla related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants