fix: yml 들여쓰기 수정

[Gyuhyeok99]

관련 이슈

• resolves: fix: yml 들여쓰기 수정 #554

작업 내용

들여쓰기 수정해도 도커 jdk버전때문에 stage 서버가 안고쳐질 거 같아서 이거만 같이 수정했습니다.
승인해주시면 바로 머지하겠습니다.

특이 사항

리뷰 요구사항 (선택)

[coderabbitai]

Walkthrough

이번 변경사항은 두 가지 주요 항목을 포함하고 있습니다.

1. Dockerfile 기본 이미지 업데이트

   • openjdk:17-jdk에서 eclipse-temurin:17-jdk로 기본 이미지 변경
   • ARG JAR_FILE, COPY, ENTRYPOINT, VOLUME 등 다른 명령어들은 변경 없음

2. 서브모듈 참조 업데이트

   • src/main/resources/secret의 서브모듈 커밋 해시 업데이트
   • ae3e90ef74f56e93be1ede280bbc5f330ca8e297에서 8300cdecaebfc28fd657064a00a44815a7bb2eee로 변경
   • 기능적 코드 변화는 없으며, 추적되는 서브모듈 커밋만 갱신

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5분

• 기본 이미지 교체는 직접적인 로직 변경이 없는 단순한 구성 수정
• 서브모듈 업데이트는 커밋 해시 갱신으로 별도 검증 필요 없음
• 변경 범위가 제한적이고 패턴이 반복적이지 않음

Suggested reviewers

• wibaek
• whqtker
• Hexeong
• lsy1307
• sukangpunch

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	PR 제목이 yml 들여쓰기 수정을 언급하지만, 실제 변경사항은 Dockerfile의 기본 이미지 변경과 서브모듈 커밋 해시 업데이트도 포함되어 있어 제목이 전체 범위를 충분히 반영하지 못합니다.	제목을 'fix: yml 들여쓰기 및 Dockerfile JDK 버전 업데이트' 또는 유사하게 수정하여 모든 주요 변경사항을 포함시키세요.
Out of Scope Changes check	⚠️ Warning	Dockerfile의 기본 이미지 변경(openjdk:17-jdk → eclipse-temurin:17-jdk)은 이슈 #554의 yml 들여쓰기 수정과 직접적으로 관련이 없어 범위를 벗어난 변경사항입니다.	Dockerfile 변경을 별도 PR 또는 이슈로 분리하거나, 이 변경이 #554의 일부임을 명확히 하는 문서를 추가하세요.
Linked Issues check	❓ Inconclusive	PR의 변경사항(yml 들여쓰기 수정과 Dockerfile JDK 버전 변경)이 링크된 이슈 #554의 명시된 목표(yml 들여쓰기 수정)를 충족하지만, Dockerfile 변경은 이슈 설명에 명시되지 않아 부분적으로만 부합합니다.	이슈 #554에서 Dockerfile JDK 버전 변경이 원래 요구사항에 포함되었는지 명확히 하거나, 해당 변경을 별도 이슈로 분리하는 것을 검토하세요.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	PR 설명이 기본 템플릿 구조를 따르고 관련 이슈(#554)를 명시했으며, 작업 내용을 간단히 설명했지만 특이 사항 및 리뷰 요구사항 섹션이 채워지지 않아 완성도가 부분적입니다.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 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 093b6af and 8ddcfb9.

📒 Files selected for processing (2)

• Dockerfile (1 hunks)
• src/main/resources/secret (1 hunks)

🔇 Additional comments (3)

src/main/resources/secret (1)

1-1: 서브모듈 참조 업데이트는 형식상 정상이지만, 검토 범위가 불완전합니다.

PR 목표와 제공된 파일을 비교하면 다음과 같은 문제가 있습니다:

1. 제공되지 않은 핵심 변경사항

   • yml 들여쓰기 수정: 제공된 파일에서 확인 불가
   • Dockerfile JDK 버전 변경 (openjdk:17-jdk → eclipse-temurin:17-jdk): 제공된 파일에서 확인 불가

2. 서브모듈 참조 변경

   • 커밋 해시 형식: ✓ 유효함
   • 변경 내용 검증 불가: 업데이트된 8300cdecaebfc28fd657064a00a44815a7bb2eee 커밋의 구체적 내용이 명확하지 않음

다음 파일들을 검토를 위해 제공해 주시기 바랍니다:

• Dockerfile (JDK 버전 변경사항 확인용)
• 수정된 yml 파일들 (들여쓰기 변경사항 확인용)

또한 업데이트된 secret 서브모듈에 yml 들여쓰기 수정이 포함되어 있다면, 그 내용을 브리핑해 주시겠어요?

Dockerfile (2)

2-2: 기본 이미지 변경: 최신 권장 사항 적용됨

기존의 openjdk:17-jdk 이미지에서 eclipse-temurin:17-jdk로 변경된 점이 좋습니다.

주요 개선사항:

1. 보안 강화 — OpenJDK 이미지는 지원이 종료되었으며, Eclipse Temurin이 공식 후속 이미지입니다
2. 장기 유지보수 용이 — 활발히 유지보수되는 이미지로 교체되어 향후 보안 업데이트 보장됨
3. 버전 일관성 — JDK 17 버전은 유지되므로 호환성에 영향 없음

---

1-14: PR 범위 확인 요청

이 PR의 제목은 "yml 들여쓰기 수정"이며, PR 설명에서도 두 가지 변경사항을 언급하고 있습니다:

1. YAML 들여쓰기 수정 (주요 변경)
2. Docker JDK 버전 수정 (추가 변경)

현재 리뷰 범위에는 Dockerfile만 포함되어 있어, YAML 파일들이 누락된 것으로 보입니다.

다음을 확인해주세요:

• YAML 들여쓰기 수정이 포함된 파일들이 리뷰 범위에 모두 포함되었는지
• 이 리뷰에 포함되어야 할 다른 파일들이 있는지

리뷰를 완료하기 위해 모든 변경 파일의 확인이 필요합니다.

---

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.}

[Hexeong]

승인했습니다~