Skip to content

fix: Tempo URL 및 Alloy config 수정 (#49)#52

Merged
doup2001 merged 2 commits intomainfrom
fix/#49
Apr 11, 2026
Merged

fix: Tempo URL 및 Alloy config 수정 (#49)#52
doup2001 merged 2 commits intomainfrom
fix/#49

Conversation

@doup2001
Copy link
Copy Markdown
Member

@doup2001 doup2001 commented Apr 11, 2026

📌 작업한 내용

  • Tempo URL 수정: Grafana Tempo 트레이싱 엔드포인트 정확한 경로 설정
  • config.alloy 수정: Alloy 관찰성 파이프라인 구성 업데이트

🔍 참고 사항

  • Tempo 트레이싱 데이터 수집 정상화 (분산 추적 완료)
  • Alloy 설정 최적화: Loki 로그 + Tempo 트레이스 + Prometheus 메트릭 통합
  • PinHouse_CLOUD 관찰성 스택 완성 (로그/메트릭/트레이스)
  • 전체 APM 체계 구축으로 디버깅 효율성 대폭 향상

🖼️ 스크린샷

  • X

🔗 관련 이슈

#49 (Grafana Loki/Tempo 설정)

✅ 체크리스트

  • 로컬에서 빌드 및 테스트 완료
  • Grafana 대시보드에서 트레이싱 확인
  • 코드 리뷰 반영 완료
  • 문서화 필요 여부 확인

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 모니터링 인프라 구성 업데이트: OTLP 트레이스 파이프라인에 gRPC 및 HTTP 지원 추가, 데이터 배치 처리 및 분산 추적 시스템으로의 내보내기 설정
    • 로그 수집 파이프라인의 레이블 지정 규칙 확장으로 서비스 및 작업 식별 개선
    • 분산 추적 조회 엔드포인트 포트 구성 업데이트

@doup2001 doup2001 self-assigned this Apr 11, 2026
@doup2001 doup2001 added the fix 기능 수정 및 버그 픽스 label Apr 11, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 11, 2026

Caution

Review failed

Pull request was closed or merged during review

개요

Alloy 모니터링에 OTLP 트레이스 파이프라인 설정 및 Loki 로그 재레이블링 규칙을 추가하고, Tempo 쿼리 프론트엔드 포트를 3100에서 3200으로 변경했습니다.

변경 사항

코호트 / 파일 요약
Alloy OTLP 및 로깅 파이프라인
k8s-helm/releases/monitoring-alloy/values.yaml
OTLP 수신기에 gRPC 및 HTTP 지원 추가, 배치 처리 및 클러스터 내 Tempo 분산기로의 내보내기 설정. Loki 재레이블링 규칙 확장: 컨테이너 이름, app 레이블, app.kubernetes.io/name에서 service 파생 추가, service_name 복사, namespace/podjob 계산. 기존 job 규칙을 정규화된 service 레이블 사용으로 변경.
Tempo 쿼리 포트 업데이트
k8s-helm/releases/monitoring-core/values.yaml
Grafana Tempo 추가 데이터 소스 URL의 포트를 3100에서 3200으로 변경.

예상 코드 리뷰 난이도

🎯 2 (Simple) | ⏱️ ~10 분

관련 PR

🐰 Alloy의 발자국이 선명해지네요,

OTLP 흐름 양쪽으로 춤을 추고,

Loki 태그들 새로 정렬되며,

Tempo의 문은 3200으로 열리고,

모니터링의 오케스트라 완성! 🎵

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 명확하게 반영하고 있습니다. Tempo URL 수정과 Alloy config 변경이라는 두 가지 주요 변경 사항을 간결하게 요약했습니다.
Description check ✅ Passed PR 설명이 템플릿의 모든 필수 섹션을 포함하고 있으며, 작업 내용, 참고 사항, 관련 이슈, 체크리스트가 충분히 작성되어 있습니다.
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/#49

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.

@doup2001 doup2001 merged commit 789b528 into main Apr 11, 2026
1 check was pending
@doup2001 doup2001 deleted the fix/#49 branch April 11, 2026 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 기능 수정 및 버그 픽스

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant