Skip to content

feat: Decouple GraalVM version from Java version in native image build - #397

Merged
EvgheniPopusoi merged 1 commit into
masterfrom
chore/choose-graal-version
Sep 8, 2026
Merged

EvgheniPopusoi merged 1 commit into
masterfrom
chore/choose-graal-version

Conversation

@EvgheniPopusoi

Copy link
Copy Markdown
Contributor

Summary

  • Adds a dedicated graalvm-version input (default 25.0) so the GraalVM release used by graalvm/setup-graalvm is pinned independently of java-version, matching the X.Y version format the action expects for GraalVM 25+ (previously java-version's major-version format, e.g. '25', was being passed into the version field).
  • Switches the native image Maven build from verify to package (-Dpackaging=native-image -DskipTests), since tests and verify-phase checks already run in the test job — avoids re-executing that work during native image packaging.

Test plan

  • Confirm the build-image job resolves and installs GraalVM 25.0 successfully for both graalvm and graalvm-community distributions
  • Confirm mvn package -Dpackaging=native-image -DskipTests still produces the native image binary (i.e. the native-image plugin binds at the package phase in caller pom.xmls)
  • Run against hiiretail-price-specification-stream-processor (the current consumer of this reusable workflow) before moving the v0 tag

…r native image

Adds a dedicated graalvm-version input (default 25.0) so the GraalVM
release used by setup-graalvm is pinned independently of java-version,
matching the X.Y version format the action expects for GraalVM 25+.
Also switches the native image Maven build from `verify` to `package`
since tests and verify-phase checks already run in the test job.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@EvgheniPopusoi
EvgheniPopusoi merged commit 52aa79e into master Sep 8, 2026
1 check passed
@EvgheniPopusoi
EvgheniPopusoi deleted the chore/choose-graal-version branch September 8, 2026 08:27
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.

2 participants