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: tracing prototype #372
Conversation
9cde1cd
to
4881600
Compare
4881600
to
270ff43
Compare
from sentry_sdk import Hub | ||
|
||
|
||
@pytest.mark.parametrize("sample_rate", [0.0, 1.0]) |
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.
Where are the test!?!??! :D
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.
soon (tm)
This reverts commit ccec6ca.
same_process_as_parent=True, | ||
sampled=None, | ||
transaction=None, | ||
op=None, |
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.
Shouldn't we make operation required?
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.
For transaction/trace spans, none of the Pytohn SDK's integrations set either OP or description. Only for child spans.
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.
If we want to make more attrs required I'd like to discuss what the difference between transaction and description is
0554455
to
60ec0cc
Compare
No idea how the event payload is supposed to look, just went by what armin patched into semaphore. We get an unlabeled event in Sentry right now