Skip to content

build: migrate from Java 17 to Java 21 - #303

Open
devin-ai-integration[bot] wants to merge 1 commit into
DevOpsfrom
devin/1788775824-java21-migration
Open

devin-ai-integration[bot] wants to merge 1 commit into
DevOpsfrom
devin/1788775824-java21-migration

Conversation

@devin-ai-integration

Copy link
Copy Markdown

Summary

Moves the build and runtime to Java 21 (Spring Boot 3.3.3 already supports it, so the parent stays put).

  • pom.xml: java.version 17 → 21, and the pinned maven-compiler-plugin 3.8.0 block with <source>/<target>1.8</source> is dropped so the plugin inherits the parent's managed version and java.version (that 1.8 setting was inconsistent with Spring Boot 3.x, which requires 17+).
  • Dockerfile: builder maven:3.8.3-openjdk-17maven:3.9-eclipse-temurin-21; runtime openjdk:17-alpineeclipse-temurin:21-jre-alpine (no openjdk:21-alpine tag exists — those images are deprecated).
  • READMEs: Jenkins prerequisite openjdk-17-jreopenjdk-21-jre.

Verified: mvn clean install -DskipTests succeeds and emits class file major version 65 (Java 21); docker build succeeds and the container reports Starting BankappApplication ... using Java 21.0.12 (it then fails on the DB connection, expected with no MySQL running).

Note: this sandbox gets HTTP 429 from Maven Central, so both builds were run through a Central mirror; that's an environment limitation, unrelated to these changes.

Link to Devin session: https://app.devin.ai/sessions/f9d9a829406b4c5cba392415c5c7bb9b
Open in Devin Desktop: https://app.devin.ai/desktop/session/f9d9a829406b4c5cba392415c5c7bb9b?variant=devin
Requested by: @joao-cognition


Note

Devin errored when opening this Pull Request as joao-cognition.
As a fallback, Devin opened this PR as itself.

Co-Authored-By: Joao Esteves <joao.esteves@cognition.ai>
@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

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.

0 participants