Skip to content

Conversation

@jeanbisutti
Copy link
Member

@jeanbisutti jeanbisutti commented Jun 17, 2025

With this PR the native tests are executed with GraalVM 24.

GraalVM native is supported by Spring Boot for version 22 or above: https://docs.spring.io/spring-boot/system-requirements.html#getting-started.system-requirements.graal

@jeanbisutti jeanbisutti requested a review from a team as a code owner June 17, 2025 08:49
@jeanbisutti jeanbisutti marked this pull request as draft June 17, 2025 08:49
@jeanbisutti jeanbisutti added the test native This label can be applied to PRs to trigger them to run native tests label Jun 17, 2025
@jeanbisutti jeanbisutti reopened this Jun 17, 2025
@jeanbisutti jeanbisutti marked this pull request as ready for review June 17, 2025 10:20
test-java-version:
- 22
- 23
- 24
Copy link
Member

Choose a reason for hiding this comment

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

do we want to say we support 23? if so, I'd suggest that we keep 23 in the matrix

Copy link
Member Author

Choose a reason for hiding this comment

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

I added 23 again but it will make the build longer.

Copy link
Member

Choose a reason for hiding this comment

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

I don't think it shouldn't make the build longer (they matrixed jobs run in parallel)

Copy link
Member Author

Choose a reason for hiding this comment

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

Matrixed jobs run in parallel but the available ressources (CPU, memory) are not infinite. Native compilation can consume significant resources.

@trask trask merged commit 0d72c47 into open-telemetry:main Jun 18, 2025
90 of 91 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test native This label can be applied to PRs to trigger them to run native tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants