Skip to content

Conversation

@devin-ai-integration
Copy link

Summary

This PR migrates the Spring Boot BankApp from Java 17 to Java 22, covering phases 1-5 of the migration plan (stopping before production deployment).

Changes made:

  • Updated pom.xml: Java version 17→22, Spring Boot 3.3.3→3.4.0, Maven Compiler Plugin 3.8.0→3.13.0, source/target 1.8→22
  • Updated Dockerfile: Build stage to maven:3.9.9-eclipse-temurin-22, runtime stage to eclipse-temurin:22-jre-alpine
  • Updated README.md and README-K8S.md: Jenkins installation instructions to use openjdk-22-jre

Validation performed:

  • Docker build completed successfully with Java 22
  • Application starts correctly (fails on MySQL connection as expected without database)

Review & Testing Checklist for Human

  • Confirm Java 22 (non-LTS) is intentional - Java 21 is the current LTS release. If long-term stability is preferred, consider using Java 21 instead.
  • Verify openjdk-22-jre package availability - The README instructions reference this package for apt install, but it may not be available in standard Ubuntu repositories since Java 22 is non-LTS. Consider updating to use Eclipse Temurin PPA or alternative installation method.
  • Test full CI/CD pipeline - Run the complete Jenkins pipeline to verify Docker image builds and deploys correctly in your environment.
  • Verify Spring Boot 3.4.0 compatibility - Review Spring Boot 3.4.0 release notes for any breaking changes affecting this application.

Recommended test plan: Deploy to a staging environment with MySQL and verify the application functions correctly end-to-end.

Notes

  • Jenkinsfiles were reviewed but contain no hardcoded Java version references (they use Docker images and shared library functions)
  • MySQL Connector 8.0.33 is compatible with Java 22

Link to Devin run: https://app.devin.ai/sessions/b34edeacd0194657a8e4f2bc822f33aa
Requested by: Satwik Bebortha (@satwik-cog)

- Update Java version from 17 to 22 in pom.xml
- Upgrade Spring Boot from 3.3.3 to 3.4.0 for Java 22 support
- Update Maven Compiler Plugin from 3.8.0 to 3.13.0
- Update source/target from 1.8 to 22 in Maven Compiler Plugin
- Update Dockerfile build stage to maven:3.9.9-eclipse-temurin-22
- Update Dockerfile runtime stage to eclipse-temurin:22-jre-alpine
- Update README.md and README-K8S.md Jenkins installation instructions

Co-Authored-By: Satwik Bebortha <satwik.bebortha@cognition.ai>
@devin-ai-integration
Copy link
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 and CI monitoring

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.

1 participant