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

KTOR-7482 Use default JDK for running tests on CI #4342

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

osipxd
Copy link
Member

@osipxd osipxd commented Sep 25, 2024

Subsystem
Tests infrastructure

Motivation
As part of KTOR-7482, we need to run tests against different JDK versions.

Solution
Use the JDK from JAVA_HOME to run tests on CI.

@osipxd osipxd self-assigned this Sep 25, 2024
@osipxd osipxd force-pushed the osipxd/run-tests-on-target-jvm branch from 689782f to 462fd68 Compare September 25, 2024 07:59
@bjhham
Copy link
Contributor

bjhham commented Sep 26, 2024

There seems to be some problems in the TLS package - we got an out-dated ECDH curve java.security.ProviderException: Curve not supported: secp128r1 (1.3.132.0.28) and some rogue reflection java.lang.reflect.InaccessibleObjectException: Unable to make private static java.time.Instant java.time.Instant.create(long,int) accessible: module java.base does not "opens java.time" to unnamed module @5b8bc155

You also seem to have unlocked a consistent failure in testHeaderIsTooLong(), so that's good 😄

@osipxd
Copy link
Member Author

osipxd commented Sep 26, 2024

Yes, finally these builds against different Java versions found something :)

@osipxd osipxd marked this pull request as draft September 26, 2024 15:23
@osipxd osipxd force-pushed the osipxd/run-tests-on-target-jvm branch from 462fd68 to 5749278 Compare September 30, 2024 10:34
@osipxd osipxd force-pushed the osipxd/run-tests-on-target-jvm branch from 5749278 to 8210bed Compare October 7, 2024 13:42
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.

2 participants