Skip to content

db 풀 문제 수정 후 배포 - #53

Merged
Sehi55 merged 2 commits into
mainfrom
dev
May 18, 2026
Merged

Sehi55 merged 2 commits into
mainfrom
dev

Conversation

@Sehi55

@Sehi55 Sehi55 commented May 18, 2026

Copy link
Copy Markdown
Contributor

📝 작업 내용

이번 PR에서 작업한 내용을 설명해주세요.
-db 풀 문제 수정 후 배포

\

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 프로덕션 배포 프로세스 안정성 강화: main 브랜치에서만 배포가 실행되도록 트리거 조건 개선
    • 모니터링 엔드포인트(상태 확인, 정보, 메트릭) 노출 설정 추가
    • 데이터베이스 연결 풀 성능 최적화를 위한 설정 추가

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7986779d-577b-470b-be75-ecf7e438938e

📥 Commits

Reviewing files that changed from the base of the PR and between 09d1613 and 69d3467.

📒 Files selected for processing (3)
  • .github/workflows/cd-prod.yml
  • src/main/resources/application-local.yml
  • src/main/resources/application-prod.yml

📝 Walkthrough

둘러보기

CD-PROD 배포 워크플로우 트리거가 main 브랜치로 제한되고, 로컬 환경에서 관리 엔드포인트(health, info, prometheus)가 노출되며, 프로덕션 데이터베이스 연결 풀 파라미터가 명시됩니다.

변경 사항

배포 및 애플리케이션 구성 업데이트

레이어 / 파일 요약
CD-PROD 배포 트리거 조건 강화
.github/workflows/cd-prod.yml
workflow_run 이벤트의 배포 조건이 CI-PROD 성공 완료와 main 브랜치 트리거를 모두 확인하도록 수정되어, 다른 브랜치에서의 우발적 배포를 방지합니다.
로컬 환경 관리 엔드포인트 노출
src/main/resources/application-local.yml
management.endpoints.web.exposure.include 설정에 health, info, prometheus를 추가하여 로컬 개발 환경에서 관리 엔드포인트에 접근 가능하도록 구성했습니다.
프로덕션 HikariCP 커넥션 풀 설정
src/main/resources/application-prod.yml
spring.datasource.hikari 블록이 추가되어 최대/최소 풀 크기(maximum-pool-size, minimum-idle)와 타임아웃/수명 파라미터(idle-timeout, max-lifetime, connection-timeout)가 명시되었습니다.

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~5 분

관련 가능성이 있는 PR

  • Miche-Let/user-service#41: 본 PR에서 강화된 cd-prod.yml workflow_run 트리거 조건(main 브랜치 필수)과 직접 관련됨
  • Miche-Let/user-service#21: 본 PR의 ci-prod 워크플로우 완료 트리거 의존성이 해당 PR의 ci-prod.yml 워크플로우 추가와 직접 연결됨
  • Miche-Let/user-service#30: 본 PR의 application-prod.yml HikariCP 설정 추가가 해당 PR의 프로덕션 데이터소스 설정과 직접 겹침

제안 라벨

enhancement

제안 리뷰어

  • githyj-jang
  • ji-circle
  • jihxonx
  • Jinyoung-Kim96
  • qldo

토끼의 축하 시

🐰 배포는 main만, 그 외에는 말고,
로컬에선 health와 prometheus 빛나고,
풀을 설정해 데이터 흐르듯,
설정들 다져진 견고한 기초!
구성의 정밀함이 빛나네~ ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 PR의 주요 변경사항을 명확히 반영하고 있습니다. 'db 풀 문제 수정 후 배포'는 HikariCP 연결 풀 설정 추가, 관리 엔드포인트 노출 설정, CD 배포 조건 강화 등의 변경사항을 잘 요약합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

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.

@Sehi55
Sehi55 merged commit dd227f3 into main May 18, 2026
3 checks passed
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.

1 participant