Skip to content

Conversation

@devin-ai-integration
Copy link

Ticket: EB-9

Changes Made

This PR modernizes the Docker base images in the Dockerfile as part of infrastructure improvements related to Epic EB-1 (Spring Boot 3.4.9 upgrade).

Specific Updates:

  1. Build Stage (Line 6): Updated from maven:3.8.3-openjdk-17 to maven:3.9.0-eclipse-temurin-17
  2. Runtime Stage (Line 28): Updated from openjdk:17-alpine to eclipse-temurin:17-jre-alpine

Benefits

  • Maven 3.9.0: Provides performance improvements and bug fixes over Maven 3.8.3
  • Eclipse Temurin: Industry-recommended OpenJDK distribution with better long-term support and reliability
  • JRE-only Runtime: Using -jre-alpine instead of full JDK reduces the final image size and attack surface
  • Smaller Runtime Base: Eclipse Temurin 17 JRE Alpine is only 181MB compared to older OpenJDK images

Testing

✅ Docker image builds successfully
✅ Build stage completes with Maven 3.9.0 and Eclipse Temurin 17
✅ Final image size: 234MB
✅ Multi-stage build process preserved
✅ Application JAR correctly copied to runtime stage

Image Size Comparison

  • Final application image: 234MB
  • Runtime base image (eclipse-temurin:17-jre-alpine): 181MB
  • Build base image (maven:3.9.0-eclipse-temurin-17): 529MB (only used during build, not in final image)

The JRE-only runtime approach provides a smaller, more secure production image while maintaining full Java 17 compatibility.


Link to Devin run: https://app.devin.ai/sessions/bc687391ae194f33937bed131312e76d
Requested by: Alex Peng (alex.peng@windsurf.com) (@alexjpeng)

…murin 17 JRE

Co-Authored-By: Alex Peng <alexander.j.peng@gmail.com>
@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