Skip to content

OpenJ9 Valhalla OpenJDK test updates#6985

Merged
JasonFengJ9 merged 1 commit intoadoptium:masterfrom
theresa-m:valhalla_updates_3
Mar 31, 2026
Merged

OpenJ9 Valhalla OpenJDK test updates#6985
JasonFengJ9 merged 1 commit intoadoptium:masterfrom
theresa-m:valhalla_updates_3

Conversation

@theresa-m
Copy link
Copy Markdown
Contributor

  • Add "_j9" to end of test names since these are specific to j9.
  • Modify hotspot_valhalla to only run runtime tests.
  • Add a seperate target for Valhalla serviceability tests.
  • Remove compiler tests from ProblemList_openjdkvalhalla-openj9.txt. These don't need to be run since they test hotspot JIT implementations.

Once this is merged I will modify https://hyc-runtimes-jenkins.swg-devops.com/view/Valhalla%20Tests/job/Test_JDKnext_valhalla_openjdk_x86-64_linux/ to run jdk_valhalla_j9,hotspot_valhalla_runtime_j9,serviceability_valhalla_j9.

fyi @hangshao0

- Add "_j9" to end of test names since these are specific to j9.
- Modify hotspot_valhalla to only run runtime tests.
- Add a seperate target for Valhalla serviceability tests.
- Remove compiler tests from ProblemList_openjdkvalhalla-openj9.txt.
These don't need to be run since they test hotspot JIT implementations.

Once this is merged I will modify https://hyc-runtimes-jenkins.swg-devops.com/view/Valhalla%20Tests/job/Test_JDKnext_valhalla_openjdk_x86-64_linux/
to run jdk_valhalla_j9,hotspot_valhalla_runtime_j9,serviceability_valhalla_j9.

Signed-off-by: Theresa Mammarella <Theresa.T.Mammarella@ibm.com>
@hangshao0
Copy link
Copy Markdown
Contributor

Remove compiler tests from ProblemList_openjdkvalhalla-openj9.txt. These don't need to be run since they test hotspot JIT implementations.

FYI @hzongaro @a7ehuo

@pshipton pshipton requested a review from JasonFengJ9 March 31, 2026 18:34
@JasonFengJ9 JasonFengJ9 merged commit 0befeb4 into adoptium:master Mar 31, 2026
3 checks passed
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