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.

Sign up for GitHub

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

Jump to bottom

Module support not available in GraalVM / Mandrel 21.3 #28475

Closed
zakkak opened this issue Oct 10, 2022 · 3 comments · Fixed by #28574
Closed

Module support not available in GraalVM / Mandrel 21.3 #28475

zakkak opened this issue Oct 10, 2022 · 3 comments · Fixed by #28574
Assignees
@zakkak
Labels
area/mandrel area/native-image kind/bug Something isn't working
Milestone
2.13.4.Final

Comments

@zakkak
Copy link
Contributor

zakkak commented Oct 10, 2022 •
edited

Describe the bug

#28409 adds support for modules. However, modules are not supported in GraalVM / Mandrel 21.3.

Expected behavior

Quarkus native integration tests should pass when using 21.3.

Actual behavior

Quarkus integartion tests fail when using 21.3. The error is:

--add-modules=jdk.net

See https://github.com/graalvm/mandrel/actions/runs/3216808337/jobs/5259259874#step:11:693

How to Reproduce?

Run quarkus integration tests with -Dquarkus.native.builder-image=quay.io/quarkus/ubi-quarkus-mandrel:21.3-java11

Output of uname -a or ver

No response

Output of java -version

11

GraalVM version (if different from Java)

21.3

Quarkus version or git rev

06c4344

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response
@zakkak zakkak added kind/bug Something isn't working area/native-image labels Oct 10, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 10, 2022

/cc @Karm, @galderz

@zakkak
Copy link
Contributor Author

zakkak commented Oct 10, 2022 •
edited

Note: This might be OK if Quarkus 2.14 gets released with 22.3 support and we decide to drop 21.3 support.

@zakkak zakkak self-assigned this Oct 12, 2022
zakkak added a commit to zakkak/quarkus that referenced this issue Oct 12, 2022
@zakkak
Don't use --add-modules with GraalVM < 22.0.0.2
d779a4a 
Support for --add-modules was added in 22.0.0.2
oracle/graal@f7b9869

Fixes quarkusio#28475
@zakkak zakkak mentioned this issue Oct 12, 2022
Closed
@jerboaa
Copy link
Contributor

jerboaa commented Oct 12, 2022

Note: This might be OK if Quarkus 2.14 gets released with 22.3 support and we decide to drop 21.3 support.

+1

zakkak added a commit to zakkak/quarkus that referenced this issue Oct 13, 2022
@zakkak
Bump minimum GraalVM version to 22.2.0
b81f1ee 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
@zakkak zakkak mentioned this issue Oct 13, 2022
Merged
zakkak added a commit to zakkak/quarkus that referenced this issue Oct 14, 2022
@zakkak
Bump minimum GraalVM version to 22.2.0
4f7b7e6 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
@quarkus-bot quarkus-bot bot added this to the 2.14 - main milestone Oct 14, 2022
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 16, 2022
@zakkak @igorregis
Bump minimum GraalVM version to 22.2.0
59d673f 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 16, 2022
@zakkak @igorregis
Bump minimum GraalVM version to 22.2.0
c2db2c8 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 16, 2022
@zakkak @igorregis
Bump minimum GraalVM version to 22.2.0
c835174 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 17, 2022
@zakkak @igorregis
Bump minimum GraalVM version to 22.2.0
a207971 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
Liuigi pushed a commit to Liuigi/quarkus that referenced this issue Oct 17, 2022
@zakkak @Liuigi
Bump minimum GraalVM version to 22.2.0
8d86363 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
tmihalac pushed a commit to tmihalac/quarkus that referenced this issue Oct 27, 2022
@zakkak @tmihalac
Bump minimum GraalVM version to 22.2.0
620b0e6 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530
@gsmet gsmet modified the milestones: 2.14.0.CR1, 2.13.4.Final Nov 2, 2022
gsmet pushed a commit to gsmet/quarkus that referenced this issue Nov 2, 2022
@zakkak @gsmet
Bump minimum GraalVM version to 22.2.0
c1eedf8 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530

(cherry picked from commit 4f7b7e6)
zakkak added a commit to zakkak/quarkus that referenced this issue Nov 15, 2022
@zakkak
Bump minimum GraalVM version to 22.2.0
6b605bb 
As discussed in quarkusio#28529

Supersedes quarkusio#28529
Closes quarkusio#28475
Relates to quarkusio#28530

(cherry picked from commit 4f7b7e6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zakkak zakkak

Labels
area/mandrel area/native-image kind/bug Something isn't working
Projects
None yet
Milestone
2.13.4.Final
3 participants
@jerboaa @gsmet @zakkak