Skip to content

fix: 런타임 버그 수정 및 성능 개선 (closes #42)#43

Merged
younchanhyeok merged 8 commits into
devfrom
fix/#42-runtime-bug-and-performance
May 7, 2026
Merged

fix: 런타임 버그 수정 및 성능 개선 (closes #42)#43
younchanhyeok merged 8 commits into
devfrom
fix/#42-runtime-bug-and-performance

Conversation

@younchanhyeok
Copy link
Copy Markdown
Contributor

관련 이슈

closes #42

변경 사항

  • TicketActionLogRepository: findByTicketIdfindByTicket_Id
  • TicketMetricSnapshotRepository: findByTicketIdfindByTicket_Id
  • TicketService: getActionLogs(), getMetricSnapshot() 메서드명 동기화
  • AnomalyDetectionService: Prometheus 중복 호출 제거 및 히스토리 조회 최적화
  • TicketService: 티켓 번호 포맷 %03d%06d, DB id 기반 생성으로 동시성 문제 해결

작업 내용 상세

🔴 런타임 버그

TicketActionLog, TicketMetricSnapshot 엔티티에 ticketId 필드가 없고 ticket 객체만 존재함에도 Repository에서 findByTicketId를 사용 → Spring Data JPA가 경로를 찾지 못해 런타임 에러 발생. findByTicket_Id로 수정하여 중첩 필드 탐색이 올바르게 동작하도록 수정.

🟡 성능 개선

  • checkPod()에서 이미 조회한 cpu/memory/errorRate를 handleAnomaly()로 전달하여 재조회 제거
  • 이상 없는 Pod는 히스토리 조회 자체를 건너뜀
  • 이상 있는 Pod의 히스토리 3개를 한 번만 조회 후 재사용 (최대 18번 → 3번)

체크리스트

  • ./gradlew build 성공 확인
  • 이상 탐지 임계치 변경이 있다면 application.yml에서만 관리되고 있음
  • 에러 응답 형식 ({ status, message, timestamp }) 준수
  • 민감 정보 (API 키, 비밀번호 등) 코드에 포함되지 않음
  • AI 서버 연동 변경 시 fallback 로직 포함 여부 확인

LEESEOJUN-creator and others added 8 commits May 7, 2026 17:04
release: dev → main
dev → main (Prometheus history range/step 수정 + 메트릭 스케줄러)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tub-main

chore: ai 패키지 불필요한 MetricPoint 스텁 삭제 (#36)
- TicketActionLogRepository: findByTicketId → findByTicket_Id
- TicketMetricSnapshotRepository: findByTicketId → findByTicket_Id
- TicketService: getActionLogs, getMetricSnapshot 메서드명 동기화
- AnomalyDetectionService: Prometheus 중복 호출 제거 및 히스토리 조회 최적화
- TicketService: 티켓 번호 포맷 %03d → %06d, DB id 기반 생성으로 동시성 문제 해결

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@younchanhyeok younchanhyeok merged commit 6345eee into dev May 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants