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

test: add native tests for GraalVM for JDK 23 #3536

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

diegomarquezp
Copy link
Contributor

This is in addition to the already existing native checks against GraalVM for JDK 21

This is part of upgrading the SDK to support GraalVM for JDK 23

This is in addition to the already existing native checks against GraalVM for JDK 21
We probably want to do this up in gax-java/gax
diegomarquezp added a commit to googleapis/sdk-platform-java that referenced this pull request Feb 7, 2025
Context: upgrading to GraalVM for JDK 23

Updating the image support in spring (GoogleCloudPlatform/spring-cloud-gcp#3536) caused errors at analysis time solved by GoogleCloudPlatform/spring-cloud-gcp@d3f6185

Gax seems to be a better place for this kind of configs.
@diegomarquezp diegomarquezp marked this pull request as ready for review February 7, 2025 20:14
@diegomarquezp diegomarquezp requested a review from a team as a code owner February 7, 2025 20:14
@diegomarquezp
Copy link
Contributor Author

The native-image.properties file must be deleted after merging googleapis/sdk-platform-java#3617 and updating this repo to the soon-to-be-released sdk-platform-java

diegomarquezp added a commit to googleapis/sdk-platform-java that referenced this pull request Feb 10, 2025
Context: upgrading to GraalVM for JDK 23

Updating the image support in spring
(GoogleCloudPlatform/spring-cloud-gcp#3536)
caused errors at analysis time solved by
GoogleCloudPlatform/spring-cloud-gcp@d3f6185

Gax seems to be a better place for this kind of configs.
…/com.google.cloud/spring-cloud-gcp-core/native-image.properties
@diegomarquezp
Copy link
Contributor Author

@lqiu96 the tests should pass after we merge the next libraries bom (contains native config for jdk 23)

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.

3 participants