feat(jdk-jtreg): run the full test/jdk inventory against Bali - #10
Open
Ivo Horák (horakivo) wants to merge 2 commits into
Open
Ivo Horák (horakivo) wants to merge 2 commits into
Ivo Horák (horakivo) wants to merge 2 commits into
Conversation
The adapter previously ran only files with a bare `@run main` (1.4k of the 12.7k files in test/jdk). It now hands jtreg every directive — `@library` via a staged `test/lib`, `@modules`, `@key`, variants, junit and testng — and excludes only the desktop, JFR, debugger and tool areas by manifest policy, which makes 7.2k files runnable. - `@requires` is evaluated through a portable `TEST.ROOT` whose property definitions come from `suites/drivers/jdk-jtreg/requires/VMProps.java`, answering upstream's names from public APIs (no WhiteBox, no HotSpot diagnostic flags), so both runtimes share one root. - The stock-JDK reference run is recorded per platform as `expectations/jdk-jtreg.reference.<platform>.json` with a fingerprint of its inputs; a matching baseline lets a run measure Bali alone. Only the linux-x64 baseline is committed; other platforms' files are git-ignored caches. - `bin/bali-docker.ts` and the release/check workflows carry the reference home and the writable `expectations/` mount needed to refresh the baseline alongside the ratchet. - The ratchet is regenerated for the new inventory; BALI.md documents the lane, its scope, and how the baseline and ratchet differ. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KDdYPYzE4xKshFSt4WGA1j
The adapter stages `suites/drivers/jdk-jtreg/requires/VMProps.java` as the portable `@requires` definitions, but the Docker plan mounted only `.harness`, `reports`, `expectations`, `registry.toml` and `manifests`; the suites tree comes from the image, which predates the driver. The Compliance / Bali job therefore failed before running a test: `ENOENT ... /work/suites/drivers/jdk-jtreg/requires/VMProps.java`. Bind the repository's `suites/drivers` over the image's, read-only. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KDdYPYzE4xKshFSt4WGA1j
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.