Skip to content

[Chore] 사용하지 않는 모니터링 스택 및 API 메트릭 제거 - #90

Merged
JangDongHo merged 3 commits into
devfrom
chore/remove-monitoring
Sep 25, 2026
Merged

JangDongHo merged 3 commits into
devfrom
chore/remove-monitoring

Conversation

@JangDongHo

@JangDongHo JangDongHo commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

📌 개요

  • 더 이상 운영하지 않는 모니터링 스택(Prometheus·Grafana)과 배포 워크플로우를 삭제했습니다.
  • 수집처가 사라진 API의 Prometheus 메트릭 수집 코드와 nginx /api/metrics 경로도 함께 제거했습니다.

✨ 작업 내용

모니터링 스택

  • monitoring/ 폴더 전체(docker-compose, nginx, prometheus, grafana 대시보드·데이터소스, letsencrypt 스크립트)를 삭제했습니다.
  • .github/workflows/monitoring-cd.yml을 삭제했습니다.

API

  • MetricsInterceptor와 PrometheusModule, 메트릭 provider(http_request_duration_seconds, http_requests_total, http_errors_total)를 제거했습니다.
  • @willsoto/nestjs-prometheus, prom-client 의존성을 제거했습니다.
  • Sentry instrument.ts에서 /api/metrics 제외 로직을 제거했습니다. 기존 tracesSampler가 그 외 요청은 1.0을 반환했으므로 tracesSampleRate: 1.0으로 바꿔도 동작은 같습니다.
  • nginx/prod.conf에서 location /api/metrics 블록을 제거했습니다. 이후 /api/metrics 요청은 /api 블록으로 넘어가 404를 반환합니다.

문서

  • README에서 Prometheus·Grafana 뱃지와 "수집 지연 감지를 위한 모니터링" 섹션을 삭제했습니다.

✅ 확인

  • pnpm build 통과
  • pnpm test 10 suites / 74 tests 통과
  • 배포 후 nginx -t 검사 통과 (CD 파이프라인)
  • 배포 후 /api/metrics가 404를 반환하는지 확인

📝 후속 작업 (저장소 외부)

  • GitHub Secrets MONITORING_SSH_HOST, MONITORING_SSH_USERNAME, MONITORING_SSH_PRIVATE_KEY 삭제
  • GCP 모니터링 서버 정리

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 변경 사항
    • Prometheus·Grafana 기반 모니터링과 관련 대시보드 및 데이터 수집 설정이 제거되었습니다. 서비스 지표와 데이터 수집 지연 현황을 확인할 수 없습니다.
    • API 요청 메트릭 수집 기능과 /api/metrics 전용 프록시 설정이 제거되었습니다.
    • 모니터링 환경을 서버에 배포하는 자동화 워크플로가 제거되었습니다.
    • README에서 모니터링 도구와 데이터 지연 사례에 관한 설명이 삭제되었습니다.

JangDongHo and others added 3 commits September 25, 2026 17:00
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: GNU-connect/Server-Node/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: bc6ddd46-92e4-475a-b19a-8f12975de93f

📥 Commits

Reviewing files that changed from the base of the PR and between 3f5eacc and 9146106.

⛔ Files ignored due to path filters (1)
  • services/api/app/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (22)
  • .github/workflows/monitoring-cd.yml
  • README.md
  • monitoring/.env.example
  • monitoring/docker-compose.dev.yml
  • monitoring/docker-compose.yml
  • monitoring/grafana/provisioning/dashboards/backend-overview.json
  • monitoring/grafana/provisioning/dashboards/dashboards.yml
  • monitoring/grafana/provisioning/dashboards/nginx-overview.json
  • monitoring/grafana/provisioning/dashboards/scraping-status.json
  • monitoring/grafana/provisioning/dashboards/supabase.json
  • monitoring/grafana/provisioning/datasources/postgres.yml
  • monitoring/grafana/provisioning/datasources/prometheus.yml
  • monitoring/letsencrypt/init-letsencrypt.sh
  • monitoring/nginx/dev.conf
  • monitoring/nginx/prod.conf
  • monitoring/prometheus/prometheus.dev.yml
  • monitoring/prometheus/prometheus.yml
  • services/api/app/package.json
  • services/api/app/src/api/common/interceptors/metrics.interceptor.ts
  • services/api/app/src/app.module.ts
  • services/api/app/src/instrument.ts
  • services/api/nginx/prod.conf
 ______________________________________________
< Plot twist: The bug was a feature all along. >
 ----------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@JangDongHo JangDongHo self-assigned this Sep 25, 2026
@JangDongHo
JangDongHo merged commit 470dc17 into dev Sep 25, 2026
1 of 2 checks passed
@JangDongHo
JangDongHo deleted the chore/remove-monitoring branch September 25, 2026 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant