Skip to content

feat: Slack 서버 에러 알림에 Grafana 로그 딥링크 추가 - #468

Merged
pooreumjung merged 2 commits into
developfrom
feat/#467-slack-grafana-log-deeplink
Sep 3, 2026
Merged

pooreumjung merged 2 commits into
developfrom
feat/#467-slack-grafana-log-deeplink

Conversation

@pooreumjung

Copy link
Copy Markdown
Member

#️⃣ Issue Number

📝 요약(Summary)

  • SlackErrorNotifier가 prod 환경에서 보내는 서버 에러 알림에 Grafana(Loki) 로그 딥링크를 추가
  • SlackMessageFormat.sendServerError()에서 요청 스레드의 MDC requestId를 읽어, {application="eatssu-{env}"} |= "reqId=<값>" LogQL로 그 요청의 로그만 필터링하는 Explore URL을 생성해 메시지 끝에 붙임
  • 시간 범위는 클릭 시점이 아니라 에러 발생 시각(절대 epoch ms) 기준 ±5분으로 고정 — 나중에 클릭해도 항상 정확한 구간이 보이도록
  • grafana.base-url/grafana.loki-datasource-uid는 환경변수(GRAFANA_BASE_URL/GRAFANA_LOKI_DATASOURCE_UID)로 주입, 미설정 시 링크 자체를 생략해 알림이 깨지지 않게 함
  • cd.yml prod 배포에 위 두 값을 GitHub Secret에서 주입하는 env 추가

💬 공유사항 to 리뷰어

  • 새 GitHub Secret 2개(EATSSU_GRAFANA_BASE_URL, EATSSU_GRAFANA_LOKI_DATASOURCE_UID) 등록 완료된 상태에서 머지해야 링크가 정상 생성됩니다. 미설정 시 조용히 생략되니 머지 자체는 안전합니다.
  • Grafana Explore URL의 panes 파라미터 구조는 Grafana의 비공식 내부 스키마라, Grafana 업그레이드 시 깨질 수 있습니다.
  • 로컬 테스트는 MySQL 없어서 못 돌렸고(Flyway 연결 실패, 환경 문제), 컴파일만 확인했습니다. CI에서 통과 여부 확인 필요.
  • 이슈 feature: Grafana Cloud Loki로 애플리케이션 로그 전송 구축 #465(Loki 로그 전송)가 prod에 아직 실제 배포되지 않았다면(현재 dev만 확인됨), 이 PR이 실사용되려면 prod에도 #465가 먼저 적용돼 있어야 합니다.

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

🤖 Generated with Claude Code

@pooreumjung pooreumjung added the feat 개발, 구현 label Sep 3, 2026
@pooreumjung pooreumjung self-assigned this Sep 3, 2026
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

Overall Project 95.59% -0.01% 🍏
Files changed 97.93% 🍏

File Coverage
SlackMessageFormat.java 99.19% -0.81% 🍏

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

Overall Project 95.59% -0.01% 🍏
Files changed 97.93% 🍏

File Coverage
JwtAuthenticationFilter.java 100% 🍏
SecurityConfig.java 100% 🍏
SlackMessageFormat.java 99.19% -0.81% 🍏

@pooreumjung
pooreumjung merged commit 3deb2e8 into develop Sep 3, 2026
1 check passed
@pooreumjung
pooreumjung deleted the feat/#467-slack-grafana-log-deeplink branch September 3, 2026 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 개발, 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: Slack 서버 에러 알림에 Grafana 로그 딥링크 추가

1 participant