Prometheus + Grafana 모니터링 구성 추가#227
Conversation
📊 코드 커버리지 리포트
|
d3a5fdc to
8cd8612
Compare
|
@minibr 모니터링 시스템 구축하시느라 고생 많으셨습니다! 적용된 Grafana 대시보드의 실제 예시 사진과 팀원들을 위한 간단한 접속/사용 방법을 PR 내용에 같이 첨부해 주시면 내용 파악에 훨씬 도움이 될 것 같습니다. 감사합니다! |
넵 확인했습니다. |
Goder-0
left a comment
There was a problem hiding this comment.
이번 모니터링 구성은 애플리케이션/도커 설정만으로 끝나지 않고, 실제 접근 경로 기준으로는 Nginx 라우팅이나 EC2 Security Group 설정까지 함께 맞아야 할 것 같은데요. 혹시 /actuator/prometheus, 9090, 3001 관련해서 외부/내부 접근 정책을 어떻게 가져갈지도 같이 정리해두신 부분이 있을까요?
예를 들어 Prometheus/Grafana를 외부에 직접 노출할지, Nginx 뒤에서 제한할지, 특정 IP만 허용할지 같은 운영 전제가 있어야 실제 적용 시 혼선이 적을 것 같아서 질문드립니다.
| # Prometheus & Grafana 실행 (이미 실행 중이면 스킵) | ||
| echo "모니터링 서비스 확인 중..." | ||
| PROMETHEUS_RUNNING=$(sudo docker ps --filter "name=prometheus" --filter "status=running" -q) | ||
| GRAFANA_RUNNING=$(sudo docker ps --filter "name=grafana" --filter "status=running" -q) | ||
|
|
||
| if [ -z "$PROMETHEUS_RUNNING" ] || [ -z "$GRAFANA_RUNNING" ]; then | ||
| echo "모니터링 서비스 시작 중..." | ||
| ${COMPOSE} up -d prometheus grafana | ||
| echo "✅ Prometheus & Grafana가 시작되었습니다." | ||
| else | ||
| echo "✅ 모니터링 서비스가 이미 실행 중입니다." | ||
| fi |
There was a problem hiding this comment.
현재 로직은 Prometheus/Grafana가 이미 실행 중이면 up -d prometheus grafana를 아예 건너뛰도록 되어 있는데요. 이번 PR처럼 prometheus.yml이나 Grafana dashboard/provisioning 파일이 변경된 경우에도, 기존 컨테이너가 떠 있으면 설정 변경이 실제 운영 환경에 반영되지 않을 가능성은 없을까요?
의도상 “최초 1회만 기동”을 생각하신 건지, 아니면 설정 파일 변경 시에도 재기동 또는 reload가 필요하다고 보신 건지 궁금합니다. 어떻게 생각하시나요?
There was a problem hiding this comment.
말씀하신 대로 현재 로직은 컨테이너가 실행 중일 때 설정 변경이 반영되지 않는 문제가 있습니다. 의도는 "최초 1회만 기동"이었으나, 이는 설정 파일 변경 시 수동 재기동이 필요한 구조라 개선이 필요했습니다. docker compose up -d는 변경사항이 없으면 컨테이너를 그대로 유지하고, 설정이 바뀐 경우에만 재생성하므로, 조건 체크를 제거하고 항상 실행하도록 수정했습니다.
좋은 지적 감사합니다. 아래와 같이 접근 정책을 정리하고 적용했습니다. /actuator/prometheus
/9090 삭제 Grafana (3001)
|
|
@minibr |
관련 이슈
PR 설명
구현 내용
수집 메트릭
리소스 설정
Grafana 대시보드
배포 준비 완료 작업
접속 방법
- Grafana: http://(EC2-IP):3001