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

5 testcases fail from extended.functional CryptoTest in JDK 24 #6019

Closed
smlambert opened this issue Mar 10, 2025 · 8 comments
Closed

5 testcases fail from extended.functional CryptoTest in JDK 24 #6019

smlambert opened this issue Mar 10, 2025 · 8 comments
Assignees

Comments

@smlambert
Copy link
Contributor

smlambert commented Mar 10, 2025

These tests are failing across all platforms in JDK 24

cryptotest/tests/KEMTests.java.KEMTests
cryptotest/tests/KeyFactoryTests.java.KeyFactoryTests
cryptotest/tests/KeyPairGeneratorTests.java.KeyPairGeneratorTests
cryptotest/tests/SignatureTests.java.SignatureTests
cryptotest/CryptoTest.java.CryptoTest

Until the real issue causing these failures can be identified and solved, these test cases should be excluded across all platforms (in prep for March JDK 24 release)

@smlambert
Copy link
Contributor Author

@zzambers - would be good to get your input on these failures

@zzambers
Copy link
Contributor

@smlambert I'll take a look

@sophia-guo sophia-guo changed the title 5 testcases fail from sanity.functional CryptoTest in JDK 24 5 testcases fail from extended.functional CryptoTest in JDK 24 Mar 10, 2025
@zzambers
Copy link
Contributor

Partially addressed by: rh-openjdk/CryptoTest#77

Other issue is addition of new kind of provider, which is not yet supported by CryptoTest.
This will need more work, see: rh-openjdk/CryptoTest#72

@smlambert smlambert moved this from Todo to In Progress in 2025 Adoptium Plan Mar 11, 2025
@zzambers
Copy link
Contributor

Fix for the other problem has also been merged: rh-openjdk/CryptoTest#78

So this should be fixed.

@smlambert
Copy link
Contributor Author

So this should be fixed.

thanks @zzambers !

Are you able to add a new tag to the CryptoTest repo?

Since we should likely start pulling from a tagged version of this repo when pulling it for use in aqa-tests, instead of master (https://github.com/adoptium/aqa-tests/blob/master/functional/security/Crypto/build.xml#L45), related: #5759.

@zzambers
Copy link
Contributor

tag: https://github.com/rh-openjdk/CryptoTest/releases/tag/v1.1

@sophia-guo
Copy link
Contributor

sophia-guo commented Mar 12, 2025

rerun failed target on linux-x64 https://ci.adoptium.net/job/Grinder/12748/ - passed
and x64 mac https://ci.adoptium.net/job/Grinder/12750/ passed
and x64 mac https://ci.adoptium.net/job/Grinder/12751

@smlambert
Copy link
Contributor Author

There are other platforms we will rerun against, but based on the 3 grinders above, mark this issue as closed/fixed. Thanks @zzambers.

@github-project-automation github-project-automation bot moved this from In Progress to Done in 2025 Adoptium Plan Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants