-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add Sentry support #108
Add Sentry support #108
Conversation
I was not able to update lock (some local issues), let's use Kebechet for that. |
Build succeeded.
|
0096ac7
to
d78bd09
Compare
@@ -85,9 +88,16 @@ def init_logging(logging_configuration: dict = None) -> None: | |||
|
|||
_init_log_levels(logging_configuration) | |||
|
|||
if _SENTRY_DSN: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any docs on this? what and why to do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the core repo.
Pipfile
Outdated
@@ -7,6 +7,7 @@ name = "pypi" | |||
"rfc5424-logging-handler" = "*" | |||
daiquiri = "*" | |||
requests = "*" | |||
raven = "*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why raven? it reads like "Raven is the legacy Python client for Sentry (getsentry.com) — replaced by sentry-python https://sentry.io"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, they are fast. That is the way I did it in the previous project :)
d78bd09
to
503de97
Compare
503de97
to
005960e
Compare
Build succeeded.
|
Depends-On: thoth-station/core#150