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.

Sign up for GitHub

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

Jump to bottom

Improve Jaeger Python traces (more spans/snapshot tag). #1845

Merged
merged 1 commit into from
Mar 18, 2024
Merged

Improve Jaeger Python traces (more spans/snapshot tag). #1845

merged 1 commit into from
Mar 18, 2024

Conversation

dom96
Copy link
Collaborator

@dom96 dom96 commented Mar 15, 2024

Adds more spans and metadata.

Tested in internal repo.

@dom96 dom96 requested review from a team as code owners March 15, 2024 20:00
@garrettgu10
Copy link
Collaborator

Not sure what's going on with CI here but lgtm

@dom96 dom96 closed this Mar 18, 2024
@dom96 dom96 reopened this Mar 18, 2024
@dom96 dom96 merged commit 85c4d35 into main Mar 18, 2024
16 checks passed
@dom96 dom96 deleted the dominik/jaeger-python-improvements branch March 18, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@garrettgu10 garrettgu10 garrettgu10 approved these changes

@hoodmane hoodmane Awaiting requested review from hoodmane hoodmane is a code owner automatically assigned from cloudflare/workers-runtime-1

@ObsidianMinor ObsidianMinor Awaiting requested review from ObsidianMinor ObsidianMinor is a code owner automatically assigned from cloudflare/workers-runtime-1

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dom96 @garrettgu10