Skip to content

fix: Tomcat Metric 수집 오류 수정 및 인증서 발급 오류 수정#30

Merged
Bumnote merged 3 commits intomainfrom
fix/grafana-monitoring
Dec 23, 2025
Merged

fix: Tomcat Metric 수집 오류 수정 및 인증서 발급 오류 수정#30
Bumnote merged 3 commits intomainfrom
fix/grafana-monitoring

Conversation

@Bumnote
Copy link
Member

@Bumnote Bumnote commented Dec 23, 2025

#️⃣ 연관된 이슈

#27

📝 작업 내용

Tomcat Metric 수집 오류 수정

  • tomcat 관련 metric을 수집하지 못하는 오류를 발견했습니다.
  • application.yml 파일에 tomcat 관련 내용을 수집할 수 있도록, tomcat 수집을 허용함으로써 오류를 해결했습니다.

새로운 도메인에 대한 인증서 발급 오류 수정

  • 기존 인증서에서 새로운 도메인을 추가하는 로직이 빠져있어 해당 내용을 추가했습니다.

Nginx Deprecated 문법 오류 수정

  • nginx 내용 중 deprecated 된 문법이 있어 해당 내용을 최신 버전으로 수정했습니다.

💬 리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요


- tomcat 관련 metric을 수집하지 못하는 오류를 발견했습니다.
- application.yml 파일에 tomcat 관련 내용을 수집할 수 있도록, tomcat 수집을 허용함으로써 오류를 해결했습니다.

issue #27
- 기존 인증서에서 새로운 도메인을 추가하는 로직이 빠져있어 해당 내용을 추가했습니다.

issue #27
- nginx 내용 중 deprecated 된 문법이 있어 해당 내용을 최신 버전으로 수정했습니다.

issue #27
@Bumnote Bumnote requested a review from SoTaeHo December 23, 2025 11:26
@Bumnote Bumnote self-assigned this Dec 23, 2025
@Bumnote Bumnote merged commit 40cc704 into main Dec 23, 2025
1 check passed
@Bumnote Bumnote linked an issue Dec 23, 2025 that may be closed by this pull request
2 tasks
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.

[Dev] Grafana 모니터링 연동

1 participant