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

Bamboo agent does not build the native image after upgrading from 3.9.1 to 3.9.2 #39879

Closed
DisterDE opened this issue Apr 4, 2024 · 12 comments
Closed
Labels
area/native-image kind/bug Something isn't working triage/duplicate This issue or pull request already exists

Comments

@DisterDE
Copy link

DisterDE commented Apr 4, 2024

Describe the bug

Hello.

The renovate bot updated the application dependency versions from 3.9.1 to 3.9.2, and the native build no longer builds.
Building two Kotlin projects (aws lambda & default web app) using Gradle 8.7 and Bamboo agents.

Build command:

./gradlew build
        -Dquarkus.package.type=native
        -Dquarkus.native.remote-container-build=true
        -Dquarkus.native.builder-image=quarkus/ubi-quarkus-mandrel-builder-image:jdk-17
        -Dnative-image.xmx=6g
        --info 
        --stacktrace

log.txt

Thanks.

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

openjdk 17.0.10 2024-01-16 LTS

Mandrel or GraalVM version (if different from Java)

MANDREL 23.0.3.0 JDK 17.0.10+7

Quarkus version or git rev

3.9.2

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

Gradle 8.7

Additional information

No response

@DisterDE DisterDE added area/native-image kind/bug Something isn't working labels Apr 4, 2024
Copy link

quarkus-bot bot commented Apr 4, 2024

/cc @Karm (mandrel), @galderz (mandrel), @geoand (kotlin), @zakkak (mandrel,native-image)

@geoand
Copy link
Contributor

geoand commented Apr 4, 2024

Thanks for reporting.

I'll let my esteemed colleagues add more, but essentially the fact that it worked previously could be considered an accident.

@zakkak @maxandersen @jponge @cescoffier another instance of GraalVM 17 :)

@jponge
Copy link
Member

jponge commented Apr 4, 2024

Does the native compilation works with a Java 21 variant?

@DisterDE
Copy link
Author

DisterDE commented Apr 4, 2024

We are using the sonar plugin and it doesn't support jdk21. So we have no possibility to work with jdk21.

@DisterDE
Copy link
Author

DisterDE commented Apr 4, 2024

But everything is okay with the 3.7.4, 3.8.3, and 3.9.1 quarkus versions. They were built successfully with the same Docker builder image and the same jdk.

@cescoffier
Copy link
Member

What @jponge suggested is to change:

        -Dquarkus.native.builder-image=quarkus/ubi-quarkus-mandrel-builder-image:jdk-17

To

        -Dquarkus.native.builder-image=quarkus/ubi-quarkus-mandrel-builder-image:jdk-21

not your Java/JVM version.

@jponge
Copy link
Member

jponge commented Apr 4, 2024

See b35ae00

@jponge
Copy link
Member

jponge commented Apr 4, 2024

@DisterDE Is there any possibility you could also check with a custom build of Quarkus on the main branch?

./mvnw -Dquickly then bump your dependencies to 999-SNAPSHOT

@zakkak
Copy link
Contributor

zakkak commented Apr 4, 2024

Looks like a duplicate of #39819

The issue should indeed be fixed on main (and is expected to be backported to 3.9).

@geoand geoand closed this as completed Apr 4, 2024
@geoand geoand added triage/duplicate This issue or pull request already exists and removed area/kotlin labels Apr 4, 2024
@zakkak
Copy link
Contributor

zakkak commented Apr 4, 2024

but essentially the fact that it worked previously could be considered an accident.

I think we should use different wording here. It didn't really work by accident as we are actually testing Quarkus with Mandrel 23.0 on a weekly base (which indeed caught the issue).

The thing is that supporting Mandrel 23.0 is not a high priority, so such failures might not get immediate attention (as it happened and 3.9.2 got out with the issue). Moving forward when using an older Mandrel version users will get a warning that will hopefully break the assumption that these versions get the same attention as the latest version, see #39866.

HTH

@geoand
Copy link
Contributor

geoand commented Apr 4, 2024

Fair enough :)

@DisterDE
Copy link
Author

DisterDE commented Apr 4, 2024

The JDK21 image works fine, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/native-image kind/bug Something isn't working triage/duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

5 participants