Conversation
WalkthroughSpring Boot Actuator를 포트 9000에 노출하도록 관리 설정을 application.yml로 중앙화하고, 개발용 application-dev.yml의 management 블록을 제거합니다. CI/CD 워크플로우(.github/workflows/dev-ci-cd.yml)의 배포 단계가 ACTUATOR_PORT(9000)를 사용하도록 환경 변수 및 헬스 체크 대상이 변경됩니다. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant CI as CI Workflow
participant Docker as Container
participant App as Spring Boot (management:9000)
CI->>Docker: 빌드 & 컨테이너 실행 (-p $ACTUATOR_PORT:9000 ...)
Docker->>App: 애플리케이션 시작 (management server on 9000)
CI->>App: GET http://localhost:$ACTUATOR_PORT/actuator/health
alt 200 OK
App-->>CI: 200 {status: "UP"}
CI-->>CI: 배포 성공 처리
else 오류/비응답
App-->>CI: 에러/타임아웃
CI-->>CI: 배포 실패 처리
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🔇 Additional comments (1)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
.github/workflows/dev-ci-cd.yml(2 hunks)src/main/resources/application-dev.yml(0 hunks)src/main/resources/application.yml(1 hunks)
💤 Files with no reviewable changes (1)
- src/main/resources/application-dev.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build-and-test
🔇 Additional comments (2)
src/main/resources/application.yml (1)
11-21: Spring Boot Actuator 설정이 올바르게 구성되었습니다.관리 포트를 9000으로 분리하고, health 및 prometheus 엔드포인트를 노출하는 설정이 적절합니다. metrics 태그도 애플리케이션 이름과 일치하도록 올바르게 설정되어 있습니다.
.github/workflows/dev-ci-cd.yml (1)
78-78: 환경 변수 설정 검증 필요Line 78에서
ACTUATOR_PORT=9000이 정의되고 Line 120에서 올바르게 사용되는 것은 확인되었습니다. 다만 위의 Docker 포트 매핑 문제가 해결된 후에 헬스 체크가 정상 작동하는지 확인하세요.Also applies to: 120-120
📝 작업 내용
Actuator의 관리 포트를 API 포트와 분리(
9000)함에 따라, CI/CD 워크플로우의 Docker 헬스체크가 실패하던 문제를 해결했습니다.헬스체크 요청이 새로운 Actuator 포트를 올바르게 바라보도록 수정하였고, 이와 함께 관련 설정의 위치를
dev프로파일에서 공통 프로파일로 이동하여 코드 중복 문제를 개선했습니다.⚡ 주요 변경사항
dev-ci-cd.yml: 헬스체크 요청 URL의 포트를 기존 API 포트에서 Actuator 전용 포트(9000)로 변경하여 배포 실패 오류를 수정했습니다.📌 리뷰 포인트
📋 체크리스트
feat: 기능1 추가)Summary by CodeRabbit