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

Quarkus 3.10.0 now requires a GraalVM installation for quarkusDev when quarkus.native.enabled is set to true #40495

Closed
ThoSap opened this issue May 7, 2024 · 2 comments · Fixed by #40496
Assignees
Labels
area/kubernetes area/native-image env/windows Impacts Windows machines kind/bug Something isn't working
Milestone

Comments

@ThoSap
Copy link

ThoSap commented May 7, 2024

Describe the bug

When upgrading Quarkus from a previous version to Quarkus 3.10.0, the Quarkus CLI migrates
quarkus.package.type=native
to

quarkus.package.jar.enabled=false
quarkus.native.enabled=true

With this change, the dev server ./gradleq quarkusDev can no longer start if any other OpenJDK distribution instead of GraalVM is installed.

With previous versions this worked, as for local development for example Eclipse Adoptium is sufficient and building and testing in native mode is only done by the CI pipeline.

Expected behavior

The local dev server ./gradleq quarkusDev should work if the following properties are set without GraalVM installed.

quarkus.package.jar.enabled=false
quarkus.native.enabled=true

Actual behavior

image

How to Reproduce?

No response

Output of uname -a or ver

Microsoft Windows [Version 10.0.22631.3447]

Output of java -version

OpenJDK 64-Bit Server VM Temurin-21.0.3+9 (build 21.0.3+9-LTS, mixed mode, sharing)

Quarkus version or git rev

3.10.0

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

Gradle 8.7

Additional information

implementation 'io.quarkus:quarkus-agroal'
implementation 'io.quarkus:quarkus-arc'
implementation 'io.quarkus:quarkus-awt'
implementation 'io.quarkus:quarkus-cache'
implementation 'io.quarkus:quarkus-config-yaml'
implementation 'io.quarkus:quarkus-jdbc-oracle'
implementation 'io.quarkus:quarkus-jdbc-postgresql'
implementation 'io.quarkus:quarkus-kubernetes'
implementation 'io.quarkus:quarkus-micrometer'
implementation 'io.quarkus:quarkus-micrometer-registry-prometheus'
implementation 'io.quarkus:quarkus-oidc'
implementation 'io.quarkus:quarkus-rest'
implementation 'io.quarkus:quarkus-rest-jackson'
implementation 'io.quarkus:quarkus-scheduler'
implementation 'io.quarkus:quarkus-smallrye-health'
implementation 'io.quarkus:quarkus-smallrye-openapi'
@ThoSap ThoSap added the kind/bug Something isn't working label May 7, 2024
@quarkus-bot
Copy link

quarkus-bot bot commented May 7, 2024

/cc @geoand (kubernetes), @iocanel (kubernetes), @zakkak (native-image)

@ThoSap ThoSap changed the title Quarkus 3.10.0 now requires a GraalVM installation when quarkus.native.enabled is set to true Quarkus 3.10.0 now requires a GraalVM installation for quarkusDev when quarkus.native.enabled is set to true May 7, 2024
@dmlloyd
Copy link
Member

dmlloyd commented May 7, 2024

I think this was indirectly caused by #39295. On reaugment, the dev mode processor forces the JAR type to be mutable-jar. Previously, this overrode the native-build setting, but now that native has its own flag, this no longer happens. It's a simple fix though.

The interim workaround would be to set quarkus.native.enabled=false while working in dev mode.

@dmlloyd dmlloyd self-assigned this May 7, 2024
dmlloyd added a commit to dmlloyd/quarkus that referenced this issue May 7, 2024
@quarkus-bot quarkus-bot bot added this to the 3.11 - main milestone May 10, 2024
@gsmet gsmet modified the milestones: 3.11 - main, 3.10.1 May 10, 2024
gsmet pushed a commit to gsmet/quarkus that referenced this issue May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubernetes area/native-image env/windows Impacts Windows machines kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants