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

Enable CryptoTests on jdk19+ #4474

Merged
merged 1 commit into from
Jun 15, 2023
Merged

Conversation

zzambers
Copy link
Contributor

CryptoTests can be enabled on jdk19+.
Issue has been fixed by: rh-openjdk/CryptoTest#31

@zzambers
Copy link
Contributor Author

@llxia
Copy link
Contributor

llxia commented Mar 28, 2023

CryptoTests did not run green for openj9. I do not think we should enable this on openj9 JDK19+ atm.
https://github.com/eclipse-openj9/openj9/issues?q=is%3Aissue+is%3Aopen+CryptoTests

@pshipton do you have any preference?

@pshipton
Copy link
Contributor

I'd prefer we don't enable Crypto tests for OpenJ9 until they are working. They fail consistently on Windows due to access denied eclipse-openj9/openj9#16710. It would be wonderful if we could disable OpenJ9 Crypto tests on Windows across all versions.

They fail sometimes on Linux machines which don't have a proper /dev/random eclipse-openj9/openj9#16720. I don't mind the occasional failure on Linux, if we enable them for Java 19 there.

@zzambers
Copy link
Contributor Author

@llxia @pshipton thanks for pointing out openj9 issues, I was not aware of some of them, I'll take a look.

@pshipton
Copy link
Contributor

I've been assuming they are OpenJ9 machine issues. We'd like to get the bad Linux machines updated, the instructions seems clear, but haven't had the resources to do it yet. For Windows, we aren't sure what we need to update on the machines to avoid the access denied problems. We've tried a couple of things, but didn't get it working.

@zzambers
Copy link
Contributor Author

zzambers commented Apr 4, 2023

CryptoTest being disabled on JDK19+ does not seem to be connected to openj9 issues.

Problems there seem to be infra related and not specific to JDK19+. (Blocking /dev/random causing timeouts, "access denied" in windows MSCAPI)

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of deleting this block of code, please leave it and add
<impl>openj9</impl>
after the <version>19+</version> section

which effectively means it will remain disabled only for that implementation, and not for hotspot implementations.

@zzambers
Copy link
Contributor Author

@smlambert Updated

JDK19 linux x86_64: OK
https://ci.adoptium.net/view/Test_grinder/job/Grinder/7293/

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.

5 participants