-
Notifications
You must be signed in to change notification settings - Fork 737
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
JEP 486: Permanently Disable the Security Manager #20563
Comments
Issue Number: 20563 |
@theresa-m Please take a look at this |
https://bugs.openjdk.org/browse/JDK-8338412 contains a list of JCL specification changes. The following classes will need to be updated in j9:
|
FYI #20586 |
#20586 unblocks the JDK24 merging & abuild jobs. |
openjdknext_j9_sanity.system_aarch64_linux
|
Thanks @JasonFengJ9 I opened adoptium/aqa-tests#5817 to remove |
I ran a grinder with adoptium/aqa-tests#5817 resulting in 32 failing sanity system tests https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/45811/. Some of these are from github.com/eclipse-openj9/openj9-systemtest.git and some from github.com/adoptium/aqa-systemtest.git. I will get in touch with the adoptium team about the best approach to fixing or disabling these. DaaLoadTest_daa1_5m |
FYI @llxia |
Most of the remaining failures are caused by a dependence on the security manager in the system test framework. I opened adoptium/STF#142. Update: I'm going to disable these tests for now since this issue needs more discussion to be resolved. |
fyi @JasonFengJ9 the failing system tests have been disabled adoptium/aqa-tests#5817 |
There are many (all?) tests failing in special.system. |
These should be fixed with adoptium/STF#143. I will run a grinder with these changes and see if any failures remain. edit: there are so many special tests my build timed out but so far I don't see any failures https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/47098/. |
I have submitted changes to remove the system test framework's dependency on the SecurityManager and re-enable the previously failing system tests for JDK 24. |
The security manager API usages within the extension repo (JDK next and 24) should be removed. |
I think the following natives Line 358 in 56b489c
Line 371 in 56b489c
and their helper method https://github.com/eclipse-openj9/openj9/blob/56b489c685f259857c9c53138e2a1a5de417bc65/runtime/j9vm/j7vmi.c#L335C1-L335C24 |
Yes we should do this as soon as we can. |
See https://openjdk.org/jeps/486
The text was updated successfully, but these errors were encountered: