Skip to content

QPPA-11633: Update the Dockerfile and DockerfileTest to use Alpine-based Maven and Eclipse Temurin Java 21 images#1629

Merged
chetanmunegowda merged 3 commits intodevelopfrom
QPPA-11633
Apr 29, 2026
Merged

QPPA-11633: Update the Dockerfile and DockerfileTest to use Alpine-based Maven and Eclipse Temurin Java 21 images#1629
chetanmunegowda merged 3 commits intodevelopfrom
QPPA-11633

Conversation

@chetanmunegowda
Copy link
Copy Markdown
Contributor

@chetanmunegowda chetanmunegowda commented Apr 28, 2026

Related Tickets & Documents

https://jira.cms.gov/browse/QPPA-11633
https://jira.cms.gov/browse/QPPA-11634


Description

  • The builder stage uses Maven to compile/package the app.
  • The final runtime stage contains only the Java 21 JRE, startup script, and built application jar.

What type of PR is this?

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore
  • ⏩ Revert

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 Confluence
  • 🙅 no documentation needed

@peter-stancic
Copy link
Copy Markdown

peter-stancic commented Apr 28, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@CMSgov CMSgov deleted a comment from Copilot AI Apr 29, 2026
@chetanmunegowda chetanmunegowda requested a review from Copilot April 29, 2026 15:24
@chetanmunegowda chetanmunegowda changed the title Update the Dockerfile and DockerfileTest to use Alpine-based Maven and Eclipse Temurin Java 21 images QPPA-11633: Update the Dockerfile and DockerfileTest to use Alpine-based Maven and Eclipse Temurin Java 21 images Apr 29, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates container build/runtime images to Alpine-based Maven + Eclipse Temurin Java 21, while also bumping Spring Framework dependencies and tightening a unit test’s cleanup behavior.

Changes:

  • Switch Dockerfile and DockerfileTest to Alpine-based Maven builder images and Alpine-based Temurin Java 21 JRE runtime images.
  • Bump Spring Framework dependencies from 6.2.17 to 6.2.18 across multiple Maven modules.
  • Add JUnit @AfterEach cleanup to reset MeasureConfigs after QualityMeasureIdEncoderTest.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
Dockerfile Replaces custom Maven install with official Maven Alpine image; switches runtime to Alpine Temurin JRE and adjusts runtime package/script handling.
DockerfileTest Moves test build/runtime images to Alpine and replaces dos2unix with sed-based script normalization.
pom.xml Updates Spring Framework BOM and related dependency versions to 6.2.18.
rest-api/pom.xml Updates Spring Framework property/BOM and explicit Spring dependency versions to 6.2.18.
acceptance-tests/pom.xml Updates Spring Web dependency version to 6.2.18.
converter/.../QualityMeasureIdEncoderTest.java Adds @AfterEach to reset shared MeasureConfigs state between tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Dockerfile
Comment thread Dockerfile Outdated
Comment thread pom.xml
nicholas-gates
nicholas-gates previously approved these changes Apr 29, 2026
@chetanmunegowda chetanmunegowda merged commit 2259b9e into develop Apr 29, 2026
6 checks passed
@chetanmunegowda chetanmunegowda deleted the QPPA-11633 branch April 29, 2026 16:35
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.

5 participants