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

feat(monitors): Add trace context to CheckIns #2241

Merged
merged 15 commits into from Jun 23, 2023
Merged

feat(monitors): Add trace context to CheckIns #2241

merged 15 commits into from Jun 23, 2023

Conversation

rjo100
Copy link
Contributor

@rjo100 rjo100 commented Jun 22, 2023

Adds contexts to the CheckIn payload in order to enable tracing. See https://github.com/getsentry/sentry-php/blob/master/src/Serializer/PayloadSerializer.php#L74 for PHP example payload

Closes getsentry/sentry#51383

@rjo100 rjo100 requested a review from a team June 22, 2023 15:58
jjbayer
jjbayer approved these changes Jun 23, 2023
View reviewed changes
relay-monitors/src/lib.rs Outdated Show resolved Hide resolved
relay-monitors/src/lib.rs Outdated Show resolved Hide resolved
rjo100 and others added 2 commits June 23, 2023 08:40
@rjo100 @jjbayer
Update relay-monitors/src/lib.rs
d16fe2b 
Co-authored-by: Joris Bayer <joris.bayer@sentry.io>
@rjo100 @jjbayer
Update relay-monitors/src/lib.rs
254da8f 
Co-authored-by: Joris Bayer <joris.bayer@sentry.io>
@rjo100 rjo100 merged commit 28c0f19 into master Jun 23, 2023
17 checks passed
@rjo100 rjo100 deleted the rjo100/trace-checkins branch June 23, 2023 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jjbayer jjbayer jjbayer approved these changes

Assignees

@rjo100 rjo100

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add trace_id to the CheckIn payload in Relay
2 participants
@rjo100 @jjbayer