-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
feat: propagate x-ray trace id to event bridge targets #12481
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
Merged
maxhoheiser
merged 24 commits into
master
from
feature/events/add-xray-trace-header-propagation
Apr 14, 2025
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
9d9be32
feat: propagate x-ray trace id
maxhoheiser fc45d72
feat: use internal trace_context and auto create segment
maxhoheiser 03dff26
feat: add x-ray trace id propagation event api gateway
maxhoheiser 1865747
feat: update test
maxhoheiser e4e8250
fix: correctly patch botocore
maxhoheiser 21c0720
feat: add test xray events lambda
maxhoheiser b29ba1d
feat: add x_ray lambda to test lambda
maxhoheiser 707d108
feat: remove xray sdk patch of already patched boto clients
maxhoheiser fdbbeab
fix: skip for v1
maxhoheiser 4e5a7aa
feat: template replace parent id
maxhoheiser bc1abe9
fix: register boto hook instead of patching all clients
maxhoheiser 786f024
fix: instrument lambda boto client call with trace header
maxhoheiser 441408a
feat: use to string from TraceHeader
maxhoheiser 921b10d
feat: validate events lambda xray test
maxhoheiser 4a83ca8
feat: validate events api gateway snapshot
maxhoheiser cfa0336
feat: switch to using x-ray trace header variable
maxhoheiser b3d5224
feat: use custom boto hook to inject trace header
maxhoheiser 2449ff5
fix: use custom input parameter and transfer to header during call
maxhoheiser 7dfd36d
feat: add test xray propagation event bridge to event bridge
maxhoheiser 00d6da4
feat: skip flake schedule rate test
maxhoheiser 1a9b072
refactor: remove unnecessary parameter
maxhoheiser 6bf8cc1
Update tests/aws/services/events/test_x_ray_trace_propagation.py
maxhoheiser 915c7d6
feat: create new trace for scheduled event
maxhoheiser 0d4c4e0
feat: unskip schedule rate test
maxhoheiser File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
question: how could those clash? could you quickly explain to me, I'm not familiar with this 😅
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.
I am piggybacking on the trace header for intermediary storing region and account for cross-region cross-account events