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

Switch graalvm CI setup to the graalvm/setup-graalvm action. #6367

Closed
wants to merge 1 commit into from

Conversation

mbien
Copy link
Member

@mbien mbien commented Aug 22, 2023

GraalVM has a new versioning scheme and the URLs changed too, lets use the opportunity to switch to the setup-graalvm action.

This will also move the minimal version to JDK 11 since graalvm CE has only support for 17 and 20. 8 and 11 appear to be already EOL and no longer downloadable, test showed that the action can also not download it. The drop of JDK 8 in the test job will also unblock PRs like #6178.

Distribution is set to the community edition.

The R component seems to be no longer available:

graalvm-community-openjdk-17.0.8+7.1]$ bin/gu available
Downloading: Component catalog from www.graalvm.org
ComponentId              Version             Component name                Stability                     Origin 
---------------------------------------------------------------------------------------------------------------------------------
espresso                 23.0.1              Java on Truffle               Supported                     github.com
espresso-llvm            23.0.1              Java on Truffle LLVM Java librSupported                     github.com
icu4j                    23.0.1              ICU4J                         Supported                     github.com
js                       23.0.1              Graal.js                      Supported                     github.com
llvm                     23.0.1              LLVM Runtime Core             Supported                     github.com
llvm-toolchain           23.0.1              LLVM.org toolchain            Supported                     github.com
native-image-llvm-backend23.0.1              Native Image LLVM Backend     Early adopter (experimental)  github.com
nodejs                   23.0.1              Graal.nodejs                  Supported                     github.com
python                   23.0.1              GraalVM Python                Experimental                  github.com
regex                    23.0.1              TRegex                        Supported                     github.com
ruby                     23.0.1              TruffleRuby                   Experimental                  github.com
visualvm                 23.0.1              VisualVM                      Supported                     github.com
wasm                     23.0.1              GraalWasm                     Experimental                  github.com

Lets check if the gh action is on the apache allow list (edit: it is).

@mbien mbien added CI continuous integration changes GraalVM [ci] enable GraalVM tests labels Aug 22, 2023
@mbien mbien force-pushed the ci-update-graal-setup branch 2 times, most recently from 7f978d6 to ef6077d Compare August 22, 2023 12:14
GraalVM has a new versioning scheme and the URLs changed too, lets
use the opportunity to switch to the setup-graalvm action.

This will also move the minimal version to JDK 11 since graalvm CE
has only support for 17 and 20. 11 appears to be already EOL.

The R component seems to be no longer available.
@mbien
Copy link
Member Author

mbien commented Aug 22, 2023

lets stash this for now graalvm/setup-graalvm#56

will open another PR which uses the old method but updates the URL

@fniephaus
Copy link

Please feel free to get in touch if you need any help with this!

@mbien
Copy link
Member Author

mbien commented Sep 20, 2023

@fniephaus thanks for the offer but I think I figured most things out by now. We stay with the old approach until the tests caught up with a supported version of GraalVM, at which point we could switch to the gh action. First step would be #6369. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI continuous integration changes GraalVM [ci] enable GraalVM tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants