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

deps: try graal-sdk 21.3.2 (old but patched) #831

Merged

Conversation

suztomo
Copy link
Contributor

@suztomo suztomo commented May 3, 2022

Trying graal-sdk 21.3.2 (old but patched), a change on top of Shubha's #827 .


Change Description

Please provide a detailed description on what changes your PR will have.

Checklist

  • Make sure to open an issue as a
    bug/issue
    before writing your code! That way we can discuss the change, evaluate
    designs, and agree on the general idea.
  • Ensure the tests and linter pass
  • Appropriate documentation is updated (if necessary)

Relevant issues:

  • Fixes #<issue_number_goes_here>

Change Description

Please provide a detailed description on what changes your PR will have.

Checklist

  • Make sure to open an issue as a
    bug/issue
    before writing your code! That way we can discuss the change, evaluate
    designs, and agree on the general idea.
  • Ensure the tests and linter pass
  • Appropriate documentation is updated (if necessary)

Relevant issues:

  • Fixes #<issue_number_goes_here>

@suztomo suztomo requested a review from a team May 3, 2022 14:28
@suztomo
Copy link
Contributor Author

suztomo commented May 3, 2022

All checks have passed. gcr.io/cloud-devrel-kokoro-resources/graalvm:latest has java11-22.0.0.2. So [kokoro] GraalVM Native / Linux actually uses GraalVM 22.0.0.2 Java 11 to run the test.

https://github.com/googleapis/testing-infra-docker/blob/c186d708c19c3b3bcfd2a0d4c9f85bc4a7b832cf/java/graalvm/Dockerfile#L15

@shubha-rajan This PR, setting the graal-sdk version 21.3.2, should work as a temporary solution against the security advisory. Mridula and I need to come up with a long-term solution.

@suztomo suztomo force-pushed the renovate/graalvm.version branch from 9232652 to eb58a2e Compare May 3, 2022 14:49
@shubha-rajan shubha-rajan merged commit 7f71d09 into GoogleCloudPlatform:main May 3, 2022
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