-
Notifications
You must be signed in to change notification settings - Fork 6.2k
8357141: Update to use jtreg 7.5.2 #25263
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
Conversation
|
👋 Welcome back cstein! A progress list of the required criteria for merging this PR into |
|
@sormuras This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be: You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 253 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details. ➡️ To integrate this PR with the above commit message to the |
Webrevs
|
|
I see a few major JEPs lined up for JDK 25 integration. I'd prefer to wait for this until after JDK 25 fork. That is, we can still put it into JDK 25 stabilization tree, just after everyone complete their integrate-merge-retest rush for JDK 25. |
|
Test runs through tier 1 to 8 show that only few unrelated tests do fail at the moment, thus the impact of updating to use jtreg 7.5.2 in the current phase is minimal. Except for provisioning the new tool version - which might put more stress to some teams compared to others. Integration after JDK 25-ea entering RDP1 is an option. |
|
Targeted to JDK 26 now. |
|
/integrate |
|
Going to push as commit fe3be49.
Your commit was automatically rebased without conflicts. |
Please review the change to update to using jtreg 7.5.2.
The primary change is to the
jib-profiles.jsfile, which specifies the version of jtreg to use, for those systems that rely on this file. In addition, therequiredVersionhas been updated in the variousTEST.ROOTfiles.Tested with tier 1 — tier 8.
Progress
Issue
Reviewers
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/25263/head:pull/25263$ git checkout pull/25263Update a local copy of the PR:
$ git checkout pull/25263$ git pull https://git.openjdk.org/jdk.git pull/25263/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 25263View PR using the GUI difftool:
$ git pr show -t 25263Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/25263.diff
Using Webrev
Link to Webrev Comment