Skip to content

prometheus histogram 메트릭 수집#246

Merged
Hwanvely merged 22 commits intomainfrom
dev
Apr 14, 2026
Merged

prometheus histogram 메트릭 수집#246
Hwanvely merged 22 commits intomainfrom
dev

Conversation

@Hwanvely
Copy link
Copy Markdown
Collaborator

No description provided.

* fix: 피버타임 검증 시 점수가 두배가 되지 않는 문제 해결

* refactor: 바뀐 dto에 맞게 BIZ 및 test 수정
* fix: 피버타임 검증 시 점수가 두배가 되지 않는 문제 해결

* refactor: 바뀐 dto에 맞게 BIZ 및 test 수정

* fix: 피버타임중 일시정지 시 피버타임도 멈추게 한다

* refactor: 인터페이스로 묶는다
* fix: 점수 증가 로직 오류 해결

* fix: 스트릭 발생 시각을 받는다

* refactor: 네이밍 수정

* fix(test): 스트릭 발생 시각 추가
* fix: 치팅 및 네트워크 지연 해결

* comment: 디버깅을 위한 로그 추가

* fix: session과 fevertime 일시정지 동기화
* fix: 치팅 및 네트워크 지연 해결

* comment: 디버깅을 위한 로그 추가

* fix: session과 fevertime 일시정지 동기화

* fix: 세션 예외 오류 해결

* feat: multipart 예외 핸들링

* refactor: CD 파이프라인 OCI로 마이그레이션

* fix: end response 분리
* fix: 치팅 및 네트워크 지연 해결

* comment: 디버깅을 위한 로그 추가

* fix: session과 fevertime 일시정지 동기화

* fix: 세션 예외 오류 해결

* feat: multipart 예외 핸들링

* refactor: CD 파이프라인 OCI로 마이그레이션

* fix: end response 분리

* fix: oci 경로 수정
* feat: drain 연결 종료 대기 추가

* feat: actuator 추가
* feat: drain 연결 종료 대기 추가

* feat: actuator 추가

* feat: grafana 환경변수 추가

* refactor: 앱 컨테이너만 교체하도록 수정
@Hwanvely Hwanvely self-assigned this Apr 14, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Spring Boot management configuration in both development and production environments to enable percentile histograms for HTTP server request metrics and add application-specific tags. A review comment points out an unnecessary space introduced in the management endpoint exposure list in the development configuration, suggesting its removal for consistency and to avoid potential parsing issues.

web:
exposure:
include: "health,prometheus,loggers"
include: "health,prometheus, loggers"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

설정 값 내의 쉼표 뒤에 불필요한 공백이 포함되어 있습니다. 다른 설정 파일(application-production.yml)과의 일관성을 유지하고 잠재적인 파싱 오류를 방지하기 위해 공백을 제거하는 것이 좋습니다.

        include: "health,prometheus,loggers"

@Hwanvely Hwanvely merged commit bc3337f into main Apr 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant