Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 575451 - [17] LeakTestSuite failing when run using Java 17
Summary: [17] LeakTestSuite failing when run using Java 17
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.21   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: BETA J17   Edit
Assignee: Noopur Gupta CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 572061
  Show dependency tree
 
Reported: 2021-08-17 07:04 EDT by Noopur Gupta CLA
Modified: 2021-09-04 12:31 EDT (History)
2 users (show)

See Also:


Attachments
flags (16.60 KB, text/plain)
2021-08-31 08:25 EDT, Noopur Gupta CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Noopur Gupta CLA 2021-08-17 07:04:23 EDT
cen64-gtk3-java17_linux.gtk.x86_64_17

https://download.eclipse.org/eclipse/downloads/drops4/Y20210813-0710/testresults/html/org.eclipse.jdt.ui.tests_ep421Y-unit-cen64-gtk3-java17_linux.gtk.x86_64_17.html

JVM settings `--add-modules ALL-SYSTEM --add-opens java.base/jdk.internal.loader=ALL-UNNAMED --add-opens jdk.localedata/sun.util.resources.cldr.provider=ALL-UNNAMED --add-opens jdk.localedata/sun.util.resources.provider=ALL-UNNAMED --add-opens java.base/jdk.internal.module=ALL-UNNAMED --add-opens java.base/java.lang.module=ALL-UNNAMED --add-opens java.base/jdk.internal.reflect=ALL-UNNAMED --add-opens java.base/jdk.internal.ref=ALL-UNNAMED --add-opens java.base/jdk.internal.math=ALL-UNNAMED --add-opens java.base/jdk.internal.misc=ALL-UNNAMED --add-opens java.base/jdk.internal.icu.impl=ALL-UNNAMED --add-opens java.base/jdk.internal.icu.text=ALL-UNNAMED --illegal-access=permit` are probably missing. Exception : Unable to make field transient java.util.HashMap$Node[] java.util.HashMap.table accessible: module java.base does not "opens java.util" to unnamed module @65a7bc80

java.lang.RuntimeException: JVM settings `--add-modules ALL-SYSTEM --add-opens java.base/jdk.internal.loader=ALL-UNNAMED --add-opens jdk.localedata/sun.util.resources.cldr.provider=ALL-UNNAMED --add-opens jdk.localedata/sun.util.resources.provider=ALL-UNNAMED --add-opens java.base/jdk.internal.module=ALL-UNNAMED --add-opens java.base/java.lang.module=ALL-UNNAMED --add-opens java.base/jdk.internal.reflect=ALL-UNNAMED --add-opens java.base/jdk.internal.ref=ALL-UNNAMED --add-opens java.base/jdk.internal.math=ALL-UNNAMED --add-opens java.base/jdk.internal.misc=ALL-UNNAMED --add-opens java.base/jdk.internal.icu.impl=ALL-UNNAMED --add-opens java.base/jdk.internal.icu.text=ALL-UNNAMED --illegal-access=permit` are probably missing.
Exception :
Unable to make field transient java.util.HashMap$Node[] java.util.HashMap.table accessible: module java.base does not "opens java.util" to unnamed module @65a7bc80
at org.eclipse.jdt.ui.leaktest.reftracker.ReferenceTracker.setAccessible(ReferenceTracker.java:113)
at org.eclipse.jdt.ui.leaktest.reftracker.ReferenceTracker.followFieldReference(ReferenceTracker.java:70)
at org.eclipse.jdt.ui.leaktest.reftracker.ReferenceTracker.visit(ReferenceTracker.java:151)
at org.eclipse.jdt.ui.leaktest.reftracker.ReferenceTracker.start(ReferenceTracker.java:175)
at org.eclipse.jdt.ui.leaktest.LeakTestCase.collect(LeakTestCase.java:58)
at org.eclipse.jdt.ui.leaktest.LeakTestCase.assertInstanceCount(LeakTestCase.java:123)
at org.eclipse.jdt.ui.tests.leaks.JavaLeakTest.internalTestEditorOpen(JavaLeakTest.java:378)
at org.eclipse.jdt.ui.tests.leaks.JavaLeakTest.testJavaEditorActionDelegate(JavaLeakTest.java:465)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
...
Comment 1 Eclipse Genie CLA 2021-08-17 08:57:48 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184144
Comment 3 Eclipse Genie CLA 2021-08-18 11:45:36 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184211
Comment 5 Eclipse Genie CLA 2021-08-20 02:26:14 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184250
Comment 7 Noopur Gupta CLA 2021-08-20 02:28:25 EDT
(In reply to Eclipse Genie from comment #1)
> New Gerrit change created:
> https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184144
For java.util.

(In reply to Eclipse Genie from comment #3)
> New Gerrit change created:
> https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184211
For java.security.

(In reply to Eclipse Genie from comment #5)
> New Gerrit change created:
> https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184250
For java.io.
Comment 8 Eclipse Genie CLA 2021-08-22 04:35:37 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184292
Comment 9 Noopur Gupta CLA 2021-08-22 04:36:53 EDT
(In reply to Eclipse Genie from comment #8)
> New Gerrit change created:
> https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184292
For java.lang.
Comment 11 Noopur Gupta CLA 2021-08-23 03:09:55 EDT
See also https://openjdk.java.net/jeps/403.

Adding individual packages doesn't seem to be the right approach.
Comment 12 Eclipse Genie CLA 2021-08-31 03:49:35 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184792
Comment 14 Noopur Gupta CLA 2021-08-31 08:25:21 EDT
Created attachment 287053 [details]
flags

(In reply to Eclipse Genie from comment #12)
> New Gerrit change created:
> https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184792
Add the above flags (shared by Kalyan).
Comment 15 Eclipse Genie CLA 2021-09-01 04:13:44 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184835
Comment 17 Eclipse Genie CLA 2021-09-02 03:49:30 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184905
Comment 19 Eclipse Genie CLA 2021-09-03 03:05:53 EDT
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/184956
Comment 21 Kalyan Prasad Tatavarthi CLA 2021-09-03 03:46:19 EDT
@kitlo Require a new Y-build to pick up these new changes.
Comment 23 Kalyan Prasad Tatavarthi CLA 2021-09-04 12:31:13 EDT
(In reply to Kit Lo from comment #22)
> Y20210903-0800 build is now available.
> 
> Eclipse downloads:
> https://download.eclipse.org/eclipse/downloads/drops4/Y20210903-0800
> 
> Build logs and/or test results (eventually):
> https://download.eclipse.org/eclipse/downloads/drops4/Y20210903-0800/
> testResults.php
> 
> Check unanticipated comparator messages:
> https://download.eclipse.org/eclipse/downloads/drops4/Y20210903-0800/
> buildlogs/comparatorlogs/buildtimeComparatorUnanticipated.log.txt
> 
> Software site repository:
> https://download.eclipse.org/eclipse/updates/4.21-Y-builds
> 
> Specific (simple) site repository:
> https://download.eclipse.org/eclipse/updates/4.21-Y-builds/Y20210903-0800
> 
> Equinox downloads:
> https://download.eclipse.org/equinox/drops/Y20210903-0800

Verified in the above build.