-
-
Notifications
You must be signed in to change notification settings - Fork 71
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
pr-tester improvements to provide more reliable and better coverage #1057
Comments
Further build test idea: Run a test to verify reproducibility of an existing GA build to ensure we haven't broken that in any way. |
Also wonder if there's any value in having another keyword trigger Also noting that it's not possible to re-run the lower level jobs e.g. https://ci.adoptium.net/job/build-scripts-pr-tester/job/build-test/job/openjdk8-pipeline/ or https://ci.adoptium.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk21u/job/jdk21u-linux-x64-temurin/ unless you change If you try to rebuild the top level PR tester in jenkins to avoid clogging up the comments in the PR you get:
|
Following a discussion and review of the current pr-tester, the following suggestions are worth consideration for improvements:
Why not build them all?
Space and time complexity
Runs with an explicit command
Saves the artifacts
We can run a set that covers the versions and platforms well, but it doesn’t need to be the entire set
The text was updated successfully, but these errors were encountered: