-
-
Notifications
You must be signed in to change notification settings - Fork 316
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
add logic for jdk-version specific suites #5575
Comments
Thanks for raising this issue @judovana. Before any option is chosen, it will be good to create a table of external apps, their versions and which ones work with which Java version. That will be useful to visualize and help inform the best way forward. |
thats terrible task.... I would much rather go one by one:( |
But yah, would try to do that. You are rigth that it is moreover necessary |
We can work on it together :) |
I will wait for #5553 to merge. Once I will be doing such a matrix, I will include multi-base-images to it. |
#5553 (comment) have reminded me old issue, where some external tests, eg jacoco, works fine with 8-17 but do not work with jdk21
Unluckily, current logic to clone and checkout is in https://github.com/adoptium/aqa-tests/blob/master/external/jacoco/test.properties
So I see several options with many pros and cons:
This will be quite a precedent for other - at least external - tests
Many others maybe have to be simply disabled for older/newer jdks, but that is easily done in playlists?
<testCase>
per set of jdks.The text was updated successfully, but these errors were encountered: