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

Feat/sentry tracing #75

Merged
merged 11 commits into from
Dec 27, 2024

Conversation

stanislaushimovolos
Copy link

@stanislaushimovolos stanislaushimovolos commented Dec 24, 2024

Adding explicit sentry-sdk integration with tracing support. Trace-id is shared across batch, each sub-response uses a separate transaction.

  1. Example of batch trace
    Screenshot from 2024-12-10 12-38-27
  2. Example of sub-response trace
    Screenshot from 2024-12-10 12-38-27

@spumer
Copy link
Collaborator

spumer commented Dec 24, 2024

I think we can add detailed info about this integration to README. @smagafurov ?

@spumer
Copy link
Collaborator

spumer commented Dec 24, 2024

Я решил не мучать нас английским, прошу прощения что не по канонам гитихаба)

try:
yield
except Exception as exc:
if isinstance(exc, BaseError):

Choose a reason for hiding this comment

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

еще хотел уточнить по поводу этой штуки. Раньше вроде тоже не логировались такие ошибки? По крайней мере я с дебагером смотрел, что оно он идет мимо логгера.
Вроде это правильное повоедение, если кто-то кинул такую ошибку, то он знает, что делает)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Да, эти ошибки превращаются в валидные ответы, например это позволяет бросать ValidationError-ы всякие, что есть вполне адекватный ответ

@spumer spumer merged commit 05f9723 into smagafurov:master Dec 27, 2024
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