Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#190] Sentry 설정 추가 #191

Merged
merged 2 commits into from
Apr 2, 2023
Merged

[#190] Sentry 설정 추가 #191

merged 2 commits into from
Apr 2, 2023

Conversation

JoosungKwon
Copy link
Member

@JoosungKwon JoosungKwon commented Apr 2, 2023

💁🏻 개요

모니터링 시스템을 구축하기 위해 빠르고 간단하게 추가 가능한 Sentry를 도입하고자 합니다.

Why?

  • 본격적인 운영에 앞서 로그 및 이슈, 에러에 대한 추적 및 대응이 현재 부족합니다.
    • Slack을 통해 로그 내용을 알림받지만 추적은 어렵습니다.
    • 시각화가 잘 안되어 있습니다.
    • 성능을 측정하기 어렵습니다.

따라서, 일단 Sentry를 도입하여 사용해본 후에 괜찮다면 그대로 사용하고 불편하다면 다른 툴을 사용하는 것을 고려하고 있습니다!!

고승원님의 추천이 있었습니다. (링크)

🌱 작업 사항

  • 의존성 추가
  • dsn key 추가
    • 환경 설정에 SENTRY_DSN 추가하고 application.yml에 dsn 키로 지정했습니다.

image

## 🦄 관련 이슈 - #190

- 의존성 추가
- dsn key 추가
@JoosungKwon JoosungKwon added the 🛠 Setting Setting label Apr 2, 2023
@JoosungKwon JoosungKwon added this to the [BE] Sprint - 4 🏃🏻 milestone Apr 2, 2023
@JoosungKwon JoosungKwon self-assigned this Apr 2, 2023
build.gradle Outdated Show resolved Hide resolved
Copy link
Collaborator

@gw0215 gw0215 left a comment

Choose a reason for hiding this comment

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

고생하셧습니다.

@JoosungKwon JoosungKwon merged commit 8812842 into develop Apr 2, 2023
@JoosungKwon JoosungKwon deleted the feat/190-apply-sentry branch April 2, 2023 10:17
@JoosungKwon JoosungKwon restored the feat/190-apply-sentry branch April 2, 2023 10:46
@JoosungKwon
Copy link
Member Author

도커 파일, 컴포즈에 내용 추가하는 걸 빠뜨려서 추가했습니다~!

fix(Docker): Sentry 환경 변수 내용 추가

@JoosungKwon JoosungKwon deleted the feat/190-apply-sentry branch August 6, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants