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

[extension/awsmiddleware] Set request ID and operation name in context. #132

Merged
merged 2 commits into from
Oct 23, 2023

Conversation

jefchien
Copy link

Description: Moves the request ID into the context and adds the operation name. Placing the metadata into the context makes it more flexible and leaves it open to more metadata in the future. This is similar to how the AWS SDKv2 sets the metadata.

Link to tracking Issue: N/A

Testing: Added unit tests.

Documentation: Updated README.

@jefchien jefchien requested a review from mxiamxia as a code owner October 23, 2023 14:23
zhihonl
zhihonl previously approved these changes Oct 23, 2023
Copy link

@zhihonl zhihonl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if relevant for this PR since we have similar errors for receiver/exporter/processor, but there's vulnerability from the PR build.

https://github.com/amazon-contributing/opentelemetry-collector-contrib/actions/runs/6614666199/job/17965125274?pr=132#step:6:201

@jefchien
Copy link
Author

Not sure if relevant for this PR since we have similar errors for receiver/exporter/processor, but there's vulnerability from the PR build.

https://github.com/amazon-contributing/opentelemetry-collector-contrib/actions/runs/6614666199/job/17965125274?pr=132#step:6:201

The govulnchecks are known to be broken. So long as it isn't breaking due to changes in this PR, I'm fine with it.

@jefchien jefchien merged commit 9bd8785 into aws-cwa-dev Oct 23, 2023
60 of 67 checks passed
@jefchien jefchien deleted the set-context-metadata branch October 23, 2023 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants