Skip to content

Conversation

@devmizz
Copy link
Contributor

@devmizz devmizz commented Jan 18, 2026

📌 Related Issue

🚨 해결하려는 문제가 무엇인가요?

⭐️ 어떻게 해결했나요?

🤔 어떤 부분에 집중하여 리뷰해야 할까요?

🗒️ 참고자료

RCA 룰

  • R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
  • C: 웬만하면 반영해 주세요. (Comment)
  • A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

Summary by CodeRabbit

  • Chores
    • 로컬 개발 및 프로덕션 환경 설정 파일 제거
    • Docker 기반 로컬 개발 환경 구성 삭제
    • Oracle 클라우드 인프라(OCI) 기반 환경 설정 추가 (데이터베이스 최적화 포함)
    • 애플리케이션 프로필 그룹 설정 업데이트

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

@devmizz devmizz self-assigned this Jan 18, 2026
@devmizz devmizz added the 🛠️ fix Refactor code label Jan 18, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

📝 Walkthrough

Walkthrough

개발 및 프로덕션 Docker 구성 파일과 환경별 애플리케이션 설정 파일을 제거하고, OCI 환경용 설정 파일에 Hikari 데이터소스 초기화 SQL을 추가했습니다. 프로필 그룹에서 dev와 prod의 console-logging 매핑도 삭제되었습니다.

Changes

Cohort / File(s) 변경 사항
Docker 구성 제거
docker/docker-compose-dev.yaml, docker/docker-compose-prod.yaml
앱 서비스 정의 및 빌드 구성 완전 삭제 (이미지, 포트 매핑, 볼륨 바인딩 포함)
Dockerfile 제거
docker/dockerfile-dev, docker/dockerfile-prod
Java 런타임 및 애플리케이션 실행 설정 완전 삭제 (베이스 이미지, WORKDIR, ENTRYPOINT 포함)
개발/프로덕션 구성 제거
src/main/resources/application-dev.yaml, src/main/resources/application-prod.yaml
환경별 설정 파일 완전 삭제 (서버, 스프링, JWT, FCM, Discord, Sentry, Kakao 설정 포함)
OCI 구성 추가
src/main/resources/application-dev-oci.yaml, src/main/resources/application-prod-oci.yaml
Hikari 연결 초기화 SQL 추가: Oracle 병렬 쿼리/DML 비활성화 설정
프로필 그룹 업데이트
src/main/resources/application.yaml
dev와 prod 프로필의 console-logging 매핑 제거

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested labels

🌏 infra

Suggested reviewers

  • eeun1225
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning PR 제목은 병렬 쿼리 옵션 설정에 관한 것이지만, 실제 변경 사항에는 Docker 파일 및 설정 파일 전체 삭제가 포함되어 있어 주요 변경 내용과 불일치합니다. 제목을 'fix: OCI 환경 병렬 쿼리 설정 및 개발/운영 설정 파일 제거'로 변경하여 주요 변경 사항(설정 파일 삭제)을 반영하시기 바랍니다.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

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

@devmizz devmizz merged commit 55181d4 into dev Jan 18, 2026
2 checks passed
@devmizz devmizz deleted the fix/attendance-multi-fix branch January 18, 2026 13:47
devmizz added a commit that referenced this pull request Jan 25, 2026
* fix: 병렬 쿼리 옵션 OCI 환경 설정에서 세팅

* infra: AWS docker-file 모두 삭제
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠️ fix Refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants