Skip to content

fix: java 버전 21을 사용하도록 통일#45

Merged
choubung merged 4 commits intodevfrom
hot-fix/ver1
Nov 18, 2025
Merged

fix: java 버전 21을 사용하도록 통일#45
choubung merged 4 commits intodevfrom
hot-fix/ver1

Conversation

@choubung
Copy link
Copy Markdown
Owner

@choubung choubung commented Nov 18, 2025

Summary by CodeRabbit

  • Chores
    • 애플리케이션 실행 환경의 Java 런타임이 버전 17에서 21로 업그레이드되었습니다(로컬 빌드, CI/CD 파이프라인 및 배포용 컨테이너 이미지 포함).
    • 최신 Java 기능 및 보안 패치가 적용되며, 최소 요구 런타임이 Java 21 이상으로 변경되어 환경 설정과 배포 구성이 갱신되어야 합니다.

@choubung choubung self-assigned this Nov 18, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 18, 2025

Walkthrough

CI/CD 워크플로우(.github/workflows/*.yml), Dockerfile 및 Gradle 설정(build.gradle)에서 Java 런타임/호환성 버전을 17에서 21로 상향 조정함.

Changes

코호트 / 파일(s) 변경 요약
CI 워크플로우
\.github/workflows/ci.yml, \.github/workflows/cd.yml
actions/setup-java@v4로 JDK 21 설정 단계 추가 및 Gradle 액션에서 명시적 Java 17 설정 제거 (빌드 단계가 새로 설정된 JDK를 사용하도록 변경)
컨테이너 기본 이미지
Dockerfile
베이스 이미지 태그 업데이트: eclipse-temurin:17.0.17_10-jre-nobleeclipse-temurin:21.0.9_10-jre-ubi9-minimal
Gradle 빌드 설정
build.gradle
Java 컴파일 호환성 변경: sourceCompatibilitytargetCompatibilityJavaVersion.VERSION_17JavaVersion.VERSION_21로 업데이트

Sequence Diagram(s)

변경이 런타임/이미지 및 빌드 설정의 버전 업그레이드에 한정되어 제어 흐름에 영향이 없으므로 시퀀스 다이어그램은 생략합니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • 추가 점검 권장 파일:
    • Dockerfile — 베이스 이미지 및 OS 변경에 따른 런타임/네이티브 라이브러리 호환성 확인
    • build.gradle — 컴파일 타겟 변경으로 인한 의존성/언어 기능 호환성 점검
    • .github/workflows/*.yml — CI 환경에서 JDK 21로 빌드/테스트가 정상 동작하는지 확인

Possibly related PRs

Poem

🐰 깡충깡충 스물하나로 뛰었네
워크플로우에 새 신발을 신기고
도커도 고운 옷으로 갈아입었지
빌드가 탕탕 통과할 때까지
나는 당근을 흔들며 춤춘다 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 풀 리퀘스트 제목이 변경 사항의 주요 내용을 명확하게 설명합니다. Java 17에서 Java 21로 통일하는 것이 모든 파일(워크플로우, Dockerfile, build.gradle)의 주요 변경 사항입니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hot-fix/ver1

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0d6bd43 and 45fb972.

📒 Files selected for processing (1)
  • .github/workflows/cd.yml (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/cd.yml

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

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ddfa728 and 2dcc0ae.

📒 Files selected for processing (3)
  • .github/workflows/cd.yml (1 hunks)
  • .github/workflows/ci.yml (1 hunks)
  • Dockerfile (1 hunks)
🔇 Additional comments (3)
.github/workflows/ci.yml (1)

28-29: Java 21 버전 업그레이드 승인

CI 워크플로우의 Java 버전을 17에서 21로 성공적으로 업그레이드했습니다. gradle-build-action@v3는 Java 21을 지원합니다.

프로젝트의 모든 테스트가 Java 21에서 정상적으로 통과하는지 확인하시기 바랍니다.

.github/workflows/cd.yml (1)

29-30: CD 워크플로우 Java 버전 통일

CD 워크플로우도 Java 21로 일관성 있게 업그레이드되었습니다. CI 워크플로우와 동일한 버전을 사용하므로 빌드 환경이 일치합니다.

Dockerfile (1)

1-1: Dockerfile 베이스 이미지 변경 확인 필요

Java 버전이 17에서 21로 업그레이드된 것은 일관적이지만, 베이스 OS가 noble(Debian 기반)에서 ubi9-minimal(RHEL 기반, 최소화 버전)으로도 변경되었습니다. 이 변경이 의도적인지 확인이 필요합니다.

검증 항목:

  • ubi9-minimal 이미지에서 애플리케이션이 정상적으로 실행되는지 확인
  • 필요한 라이브러리가 모두 포함되어 있는지 검증

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c5f82cc and 0d6bd43.

📒 Files selected for processing (2)
  • .github/workflows/cd.yml (1 hunks)
  • .github/workflows/ci.yml (1 hunks)
🔇 Additional comments (2)
.github/workflows/ci.yml (1)

23-27: Java 21 설정이 올바르게 구성되었습니다.

actions/setup-java@v4를 사용하여 JDK 21을 설정한 방식이 표준적이고 정확합니다.

.github/workflows/cd.yml (1)

23-27: Java 21 설정이 올바르게 구성되었습니다.

actions/setup-java@v4를 사용하여 JDK 21을 설정한 방식이 표준적이고 정확합니다. ci.yml과 일관성 있게 구성되었습니다.

@choubung choubung merged commit 7c3f7ff into dev Nov 18, 2025
2 checks passed
@choubung choubung linked an issue Nov 19, 2025 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

CI/CD: 배포

1 participant