jdk-version-updated

[yulakhraj]

#Jdk version mentioned is not available - (openjdk:17-jdk-alpine image is no longer available.)

Summary by CodeRabbit

• Chores
  • Updated Docker base image to improve application stability and compatibility.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The Dockerfile base image was updated from OpenJDK to Eclipse Temurin, both using Java 17 on Alpine Linux. This change switches the Docker runtime base image provider while maintaining compatibility and the same Java version.

Changes

Cohort / File(s)	Summary
Docker Base Image Update Dockerfile	Updated FROM instruction from openjdk:17-jdk-alpine to eclipse-temurin:17-jdk-alpine. All other build, copy, and run steps remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A single line, so clean and bright,
From OpenJDK to Temurin's light,
Java seventeen hops along,
Alpine base keeps us strong,
One little change makes the image right! 🏗️

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'jdk-version-updated' accurately reflects the main change in the changeset—updating the Docker base image from an unavailable openjdk:17 to eclipse-temurin:17.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

Dockerfile (1)

2-2: Consider using a more specific tag for reproducibility.

While 17-jdk-alpine works, it's a floating tag that pulls the latest patch version of Java 17. For better reproducibility and to prevent unexpected changes, consider pinning to a specific patch version.

🔎 Suggested improvement with specific version pinning

First, check the available tags to find the latest specific version:

#!/bin/bash
# Find the latest Java 17 Alpine-based tag
curl -s "https://hub.docker.com/v2/repositories/library/eclipse-temurin/tags?page_size=100&name=17" | jq -r '.results[] | select(.name | contains("jdk-alpine")) | .name' | grep -E '^17\.[0-9]+\.[0-9]+.*-jdk-alpine$' | sort -V | tail -5

Then update the Dockerfile with a specific version (example using a hypothetical version):

-FROM eclipse-temurin:17-jdk-alpine
+FROM eclipse-temurin:17.0.13_11-jdk-alpine

This ensures consistent builds across environments and makes it explicit which Java version you're using.

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ffedea0 and baebf6a.

📒 Files selected for processing (1)

• Dockerfile

🔇 Additional comments (1)

Dockerfile (1)

2-2: Excellent fix for the deprecated OpenJDK image.

Eclipse Temurin is the official replacement for the deprecated openjdk Docker images. This change maintains Java 17 compatibility while switching to a supported base image maintained by the Adoptium project (Eclipse Foundation). The eclipse-temurin:17-jdk-alpine image is publicly available and actively maintained on Docker Hub.