Skip to content

fix: 프로메테우스 exposure추가 cd 수정 - #15

Merged
Sehi55 merged 1 commit into
mainfrom
fix/14-actuator-prod-setting
May 18, 2026
Merged

Sehi55 merged 1 commit into
mainfrom
fix/14-actuator-prod-setting

Conversation

@Sehi55

@Sehi55 Sehi55 commented May 18, 2026

Copy link
Copy Markdown
Contributor

📝 작업 내용

이번 PR에서 작업한 내용을 설명해주세요.

  • 프로메테우스 exposure추가 cd 수정

🚀 주요 변경 사항

완료한 이슈 번호
Close #14
관련된 이슈 번호 (닫고 싶지 않은 경우)
Related to #

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 배포 파이프라인의 조건 처리를 최적화했습니다.
    • 프로덕션 환경의 모니터링 엔드포인트를 확장하여 관찰성을 개선했습니다.

Review Change Stack

@Sehi55 Sehi55 self-assigned this May 18, 2026
@Sehi55 Sehi55 linked an issue May 18, 2026 that may be closed by this pull request
@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown
📝 Walkthrough

개요

두 가지 프로덕션 환경 설정이 업데이트되었습니다: 애플리케이션 설정에서 프로메테우스 메트릭 엔드포인트 노출이 추가되었고, CD-PROD 배포 워크플로우는 단일 줄 조건식으로 정리되면서 workflow_run 페이로드 디버그 스텝이 추가되었습니다.

변경 사항

프로덕션 인프라 구성

계층 / 파일 요약
프로메테우스 엔드포인트 웹 노출
src/main/resources/application-prod.yml
운영 환경 actuator의 management.endpoints.web.exposure.include 설정에 prometheus가 추가되어 프로메테우스 메트릭 수집이 가능해집니다.
배포 워크플로우 조건 정리 및 디버그 로깅
.github/workflows/cd-prod.yml
deploy 잡의 조건식이 단일 줄로 정리되고, workflow_run 컨텍스트 필드(event_name, workflow_run.event, workflow_run.head_branch, workflow_run.conclusion, workflow_run.name)를 출력하는 디버그 스텝이 추가됩니다.

코드 리뷰 예상 시간

🎯 2 (Simple) | ⏱️ ~8분

토끼의 축시

🐰 프로메테우스, 이제 넌 내 친구야,
운영 서버에서 반짝반짝 빛나.
배포 로그도 선명하게 보이고,
모니터링 시작, 새로운 시대!

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning CD 워크플로우 파일의 변경사항은 이슈 #14의 범위를 벗어난 것으로 보이며, PR의 주요 목표인 Prometheus exposure 추가와는 직접적인 관련이 없습니다. CD 워크플로우 변경사항(.github/workflows/cd-prod.yml)을 별도의 PR로 분리하여 이슈 #14의 범위에 맞춘 변경사항만 포함하시기 바랍니다.
Title check ❓ Inconclusive PR 제목이 Prometheus exposure 추가라는 주요 변경사항을 포함하나, 'cd 수정'이라는 모호한 표현으로 인해 구체성이 부족합니다. 제목을 'fix: 운영환경 Prometheus actuator exposure 추가'로 변경하여 더 명확하고 구체적인 내용을 전달하시기 바랍니다.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed 변경사항이 이슈 #14의 모든 요구사항을 충족합니다. application-prod.yml에서 prometheus를 exposure 목록에 추가하여 Prometheus 404 오류를 해결했습니다.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/14-actuator-prod-setting

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/main/resources/application-prod.yml`:
- Line 22: application-prod.yml currently exposes Prometheus via the "include:
health,info,prometheus" setting which leaves /actuator/prometheus unprotected;
fix this by either adding Spring Security to the project and configuring access
control for the /actuator/prometheus endpoint (e.g., restrict to internal
roles/IPs) or by removing "prometheus" from the include list and
documenting/implementing explicit infra-level restrictions (firewall, network
policy or API gateway) to ensure /actuator/prometheus is accessible only from
trusted sources; update application-prod.yml and your project
dependency/configuration accordingly and add a short note in deployment docs
about the chosen protection method.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 24dc755e-e102-44c4-8ba0-d72fc41ace11

📥 Commits

Reviewing files that changed from the base of the PR and between 070ef2c and 2102717.

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

Comment thread src/main/resources/application-prod.yml
@Sehi55
Sehi55 merged commit c9adfd9 into main May 18, 2026
4 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.

[BUG] 운영환경 actuator 설정 오류 수정

1 participant