-
Notifications
You must be signed in to change notification settings - Fork 139
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
[24] Monitor failures in tests of Y-Builds #3228
Comments
Is this something that happened before for previous Java versions? Perhaps api tools simply can't read class files of Java 24?
|
For monitoring only jdt.core it is easier to just look at this job: |
Much of this was resolved, I assume by changes in jdt.lauching. |
We have a drop in number of tests, see https://ci.eclipse.org/releng/job/YPBuilds/job/ep434Y-unit-cen64-gtk3-java24/14/testReport/org.eclipse.jdt.core.tests.compiler.parser/TestAll/ which, however can be explained by the removal of SelectionParserTest12.java via #3264 |
While JDT/Core's BETA_JAVA24 build is green, Y-Build tests on Java 24 abort since 2024-12-31. Error is:
@MohananRahul are you aware of this problem? |
Looks like your change in #3517 really fixed the test result collection as you described in eclipse-platform/eclipse.platform.releng.aggregator#2625 (comment) 🎉 |
Indeed we have fresh test results! :) Meanwhile the following tests in jdt.core fail (while locally and during JDT-builds they pass):
|
All of them appear to be related to modules. |
Bingo: I can locally reproduce using temurin 24+30 JDK (but not using openjdk 24+29) |
@jarthana Java9ElementTests.test515342b (and possibly others) explicitly tries to read jdk/jmods/java.base.jmod, but the temurin JDK used during production builds does not have any jmods. https://bugs.eclipse.org/bugs/show_bug.cgi?id=515342 doesn't mention jmods, so do you by any chance remember (after more than 7 years ...) why the test was written like this? |
@MohananRahul @HannesWell could we switch to running tests on openjdk ea 24, rather than the temurin flavor? |
Yes absolutely. Anything that can be downloaded via a URL, preferably in tar.gz format can be used. Can you provide the URL? |
This may have been some leftover from the effort of bug 490103 that tried to get rid of the bogus jrt-fs.jar that represents the JRE container. But in the end, we decided it depended too much on the implementation details of the JDK. |
Trying to refresh memory of old discussions I came across https://bugs.eclipse.org/bugs/show_bug.cgi?id=490103#c65 Do you think we should remove the implementation for jmod support or only disable / remove those tests? |
eclipse-platform/eclipse.platform.releng.aggregator#2738 did the trick: As of https://ci.eclipse.org/releng/job/YPBuilds/job/ep435Y-unit-linux-x86_64-java24/26/testReport/ jdt.core is clean again! :) |
Listing sibling issues for convenience:
|
In particular this job should be closely monitored for [24] work:
Unfortunately there's a HUGE number of failures outside JDT (mostly against pde.api.tools), but when you scroll down to the per-package statistics, it's possible to check just everything in org.eclipse.jdt.core*.
Status as of https://ci.eclipse.org/releng/job/YPBuilds/job/ep434Y-unit-cen64-gtk3-java24/7/testReport/
The text was updated successfully, but these errors were encountered: