Skip to content

fix: Tempo local-blocks 추가로 메트릭 출력 활성화 (#49)#55

Merged
doup2001 merged 1 commit intomainfrom
fix/#49
Apr 11, 2026
Merged

fix: Tempo local-blocks 추가로 메트릭 출력 활성화 (#49)#55
doup2001 merged 1 commit intomainfrom
fix/#49

Conversation

@doup2001
Copy link
Copy Markdown
Member

@doup2001 doup2001 commented Apr 11, 2026

📌 작업한 내용

  • Tempo 설정에 local-blocks 활성화로 메트릭 생성 지원
  • 트레이스 데이터에서 자동으로 메트릭 생성 및 Prometheus 연동

🔍 참고 사항

  • metrics_generator: {} 활성화로 Span 메트릭 자동 생성 (RPM, Latency, Error Rate)
  • Grafana 대시보드에서 서비스별 트레이스 메트릭 실시간 확인 가능
  • Alloy 파이프라인과 연동하여 로그→메트릭→트레이스 완전 통합
  • PinHouse_CLOUD 관찰성 완성: Loki 로그 + Tempo 트레이스 + Prometheus 메트릭

🖼️ 스크린샷

🔗 관련 이슈

#49 (Grafana Loki/Tempo 설정 개선)

✅ 체크리스트

  • 로컬에서 빌드 및 테스트 완료
  • Grafana에서 Tempo 메트릭 생성 확인
  • Prometheus 연동 및 쿼리 정상화
  • 코드 리뷰 반영 완료

Summary by CodeRabbit

개선 사항

  • Improvements
    • Tempo 메트릭 생성이 확장되어 추가 메트릭 프로세서가 활성화되었습니다.
    • 로컬 메트릭 데이터가 스토리지에 저장되도록 설정되어 TraceQL 드릴다운 쿼리가 더 포괄적인 데이터를 활용할 수 있습니다.

@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

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 40af4f7a-3d50-4f54-96b0-4a2ca9c96dd1

📥 Commits

Reviewing files that changed from the base of the PR and between 5844455 and 2b530b5.

📒 Files selected for processing (1)
  • k8s-helm/releases/monitoring-tempo/values-prod-gitops.yaml

구현 방식

Tempo 메트릭 생성을 확장하기 위해 기본 메트릭 프로세서 설정을 수정하여 span-metricslocal-blocks를 추가하고, 해당 프로세서의 저장소 플러시 기능을 활성화했습니다.

변경 사항

Cohort / File(s) Summary
Tempo 메트릭 생성 구성
k8s-helm/releases/monitoring-tempo/values-prod-gitops.yaml
메트릭 프로세서 목록을 span-metrics에서 span-metrics, local-blocks로 확장하고, local_blocks.flush_to_storage: true를 추가하여 저장소 기반 TraceQL 메트릭 조회를 활성화했습니다.

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~3 minutes

관련 PR

🐰 로컬 블록이 저장소로 흘러가고,
메트릭들은 반짝반짝 춤을 춘다네,
TraceQL은 더 깊은 곳을 들여다보며,
모니터링의 눈이 더욱 밝아지리,
Tempo의 정박지에서 새 시대를 만난다! 📊✨

✨ 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 10c7bb4 into main Apr 11, 2026
1 check was pending
@doup2001 doup2001 deleted the fix/#49 branch April 11, 2026 07:03
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