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

PI-2342 Propagate trace context over SNS for distributed tracing #3995

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

marcus-bcl
Copy link
Contributor

@marcus-bcl marcus-bcl commented Jul 5, 2024

  • On publishing, sets the "traceparent" and optionally the "tracestate" as message attributes.
  • On consuming, starts a new span using the trace details from the message attributes.

Example showing prison identifier publishing a domain event that is consumed by custody status + key dates:
image
(App Insights link)

@marcus-bcl marcus-bcl requested a review from a team as a code owner July 5, 2024 16:05
@github-actions github-actions bot added the enhancement New feature or request label Jul 5, 2024
@marcus-bcl marcus-bcl changed the title PI-2342 Add traceId to SNS messages for distributed tracing PI-2342 Propagate trace context over SNS for distributed tracing Jul 8, 2024
@marcus-bcl marcus-bcl force-pushed the PI-2342-distributed-tracing branch from 69a9c78 to d7d0769 Compare July 8, 2024 12:03
@marcus-bcl marcus-bcl marked this pull request as ready for review July 8, 2024 12:05
@marcus-bcl marcus-bcl merged commit 0a5f52e into main Jul 8, 2024
58 of 59 checks passed
@marcus-bcl marcus-bcl deleted the PI-2342-distributed-tracing branch July 8, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants