Skip to content
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

Exclude jdk_security4 test on jdk8 linux #5220

Merged
merged 2 commits into from
Apr 17, 2024

Conversation

Haroon-Khel
Copy link
Contributor

This test is excluded on jdk11

sun/security/krb5/auto/rcache_usemd5.sh https://bugs.openjdk.java.net/browse/JDK-8258855 linux-all
due to known upstream issue.

We're seeing it on jdk8 in the dry run #5213 (comment)

Will merge this into the release branch when approved

@Haroon-Khel Haroon-Khel requested review from smlambert and sophia-guo and removed request for smlambert April 15, 2024 11:14
@smlambert
Copy link
Contributor

Happy to approve the exclusion.

Would also like some additional info, https://bugs.openjdk.org/browse/JDK-8258855 mentions 2 other testcases but not the one being excluded, would be good to confirm/document the original exclusion and this one fails in the same way (with the same exception)

From JDK-8258855

java.lang.Exception
at ReplayCacheTestProc.main0(ReplayCacheTestProc.java:278)
at ReplayCacheTestProc.main(ReplayCacheTestProc.java:325)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:78)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:567)
at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
at java.base/java.lang.Thread.run(Thread.java:831)

Noting that https://bugs.openjdk.org/browse/JDK-8262033 was closed as won't fix. If that is true, what does that mean to us, are the implications that we'd be excluding permanently.

@Haroon-Khel
Copy link
Contributor Author

Closing as it looks like an infra problem. Its passing on some machines https://ci.adoptium.net/job/Grinder/9602/ and failing on others https://ci.adoptium.net/job/Grinder/9603/testReport/. Will continue this in an infra issue

@Haroon-Khel
Copy link
Contributor Author

Infra issue raised here adoptium/infrastructure#3521

@smlambert
Copy link
Contributor

And yes, thanks once this is merged, it can be cherrypicked to the v1.0.1-release branch.

@llxia llxia merged commit 1ef8a6f into adoptium:master Apr 17, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants