Skip to content

chore: 모니터링 스택 제거#557

Merged
dh2906 merged 3 commits intodevelopfrom
chore/remove-observability-stack
Apr 16, 2026
Merged

chore: 모니터링 스택 제거#557
dh2906 merged 3 commits intodevelopfrom
chore/remove-observability-stack

Conversation

@dh2906
Copy link
Copy Markdown
Contributor

@dh2906 dh2906 commented Apr 16, 2026

🔍 개요

  • 사용하지 않기로 한 OpenTelemetry 및 Prometheus/Grafana/Loki 기반 모니터링 구성을 저장소에서 제거합니다.
  • /actuator/health 기반 컨테이너 healthcheck 안정성을 위해 actuator 자체는 유지하고, Prometheus 노출과 별도 monitoring 스택만 정리했습니다.

🚀 주요 변경 내용

  • Dockerfile, .env.example, stage/prod 배포 워크플로우에서 OpenTelemetry Java agent 배선과 OTEL 환경변수 예시를 제거했습니다.
  • build.gradle의 Prometheus registry 의존성과 application-monitoring.yml import를 제거하고, 요청 로깅 제외 패턴은 application.yml로 이동했습니다.
  • deploy-monitoring.ymlmonitoring/ 하위 Prometheus/Grafana/Loki/Promtail 배포 자산을 제거했습니다.

💬 참고 사항

  • ./gradlew test -x checkstyleMain -x checkstyleTest, ./gradlew bootJar로 변경 영향 범위를 확인했습니다.
  • ./gradlew build는 이번 변경과 무관한 기존 test checkstyle 위반 14건 때문에 실패했습니다.
  • 서버에 이미 배포된 monitoring 컨테이너/볼륨은 이 PR만으로 정리되지 않으므로 별도 운영 정리가 필요합니다.

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

dh2906 added 3 commits April 16, 2026 21:55
- Docker 이미지와 stage/prod 배포 워크플로우에서 OTel Java agent 의존성을 함께 제거해 런타임 추적 구성을 저장소 밖으로 정리한다.
- CI 빌드가 더 이상 agent 캐시, checksum, 런타임 전용 환경변수 해제 흐름에 기대지 않도록 바꿔 이미지 빌드 경로를 단순하게 유지한다.
- Prometheus registry 의존성과 별도 monitoring 설정 파일을 제거해 애플리케이션이 더 이상 메트릭 수집 구성을 내장하지 않도록 정리한다.
- Docker healthcheck가 여전히 /actuator/health 를 사용하므로 actuator 는 유지하고, 배포 안정성에 직접 영향이 큰 health 경로만 남긴다.
- Prometheus, Grafana, Loki, Promtail 배포 워크플로우와 compose 자산을 함께 제거해 저장소가 더 이상 별도 모니터링 스택을 관리하지 않도록 정리한다.
- monitoring 전용 env 예외와 타깃 갱신 스크립트까지 함께 삭제해 남은 운영 진입점을 줄이고 제거 범위를 한 커밋 안에 묶는다.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

Warning

Rate limit exceeded

@dh2906 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 28 minutes and 9 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 28 minutes and 9 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 5341a6a2-6c38-472c-bceb-0ade4773b97e

📥 Commits

Reviewing files that changed from the base of the PR and between 1029a93 and d8d4479.

📒 Files selected for processing (16)
  • .dockerignore
  • .env.example
  • .github/workflows/deploy-monitoring.yml
  • .github/workflows/deploy-prod.yml
  • .github/workflows/deploy-stage.yml
  • .gitignore
  • Dockerfile
  • build.gradle
  • monitoring/.env.example
  • monitoring/docker-compose.yml
  • monitoring/loki.yml
  • monitoring/prometheus.yml
  • monitoring/promtail.yml
  • monitoring/scripts/update-backend-active-target.sh
  • src/main/resources/application-monitoring.yml
  • src/main/resources/application.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-observability-stack

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.

@dh2906 dh2906 self-assigned this Apr 16, 2026
@dh2906 dh2906 added the 배포 운영 서버의 배포 관련 이슈입니다. label Apr 16, 2026
@dh2906 dh2906 changed the title Remove observability stack chore: 모니터링 스택 제거 Apr 16, 2026
@github-actions
Copy link
Copy Markdown

🧪 JaCoCo Coverage Report (Changed Files)

이 PR에서 변경된 main Java 소스 파일이 없습니다.

📊 View Workflow Run

@dh2906 dh2906 merged commit a9f5e01 into develop Apr 16, 2026
5 checks passed
@dh2906 dh2906 deleted the chore/remove-observability-stack branch April 16, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

배포 운영 서버의 배포 관련 이슈입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant